-
Notifications
You must be signed in to change notification settings - Fork 7
sync master with stable #34
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Conversation
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
Add timeSinceLastError and timeSinceLastAdError attributes to DATAMODEL.md
Fix: timeSinceLastError and timeSinceLastAdError Added
NR-480639
NR-480639
NR-480639
NR-480639-add-unit-tests
| name: Run Tests with Coverage | ||
| runs-on: ubuntu-latest | ||
|
|
||
| strategy: | ||
| matrix: | ||
| node-version: ['lts/*'] | ||
|
|
||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v4 | ||
|
|
||
| - name: Setup Node.js ${{ matrix.node-version }} | ||
| uses: actions/setup-node@v4 | ||
| with: | ||
| node-version: ${{ matrix.node-version }} | ||
| cache: 'npm' | ||
|
|
||
| - name: Install dependencies | ||
| run: npm ci | ||
|
|
||
| - name: Run tests with coverage | ||
| run: npm test |
Check warning
Code scanning / CodeQL
Workflow does not contain permissions Medium test
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI about 1 month ago
To fix this issue, explicitly declare a permissions block at the most appropriate scope. Since the workflow only reads code and does not require write access, set permissions: contents: read. This can be added at the workflow root (to apply to all jobs) or at the job level (to restrict just this job). Here, add the block at the job level (under test:) for the smallest change, but it is equally correct to do so at the top level. Insert this block directly beneath the name: Run Tests with Coverage line, ensuring no functionality changes. No further imports or definitions are required.
-
Copy modified lines R15-R16
| @@ -12,6 +12,8 @@ | ||
| jobs: | ||
| test: | ||
| name: Run Tests with Coverage | ||
| permissions: | ||
| contents: read | ||
| runs-on: ubuntu-latest | ||
|
|
||
| strategy: |
mlychndnnr
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
fix: npm latest releases workflow updated
Nr 425132 update pricing doc
No description provided.