-
Notifications
You must be signed in to change notification settings - Fork 48
Fixed some linter warnings #84
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
Merged
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
BUILD_ID: 20250409.0719 Signed-off-by: Lee Chee Yang <[email protected]>
- Updated to the latest `publish-docs` in `orch-ci`
* Update hash reference in publish docs workflow - Updated to the latest `publish-docs` in `orch-ci` * Update publish-docs.yml
* Update hash reference in publish docs workflow - Updated to the latest `publish-docs` in `orch-ci` * Update publish-docs.yml * Update publish-docs.yml Updated permissions * Update publish-docs.yml Pin to latest SHA * Adds branch pattern 3.0 * Update publish-docs.yml * Change working dir
* updated EMT-D overview documentation * fixed typo
* Update sb-howto.md * Update sb-howto.md * Update sb-howto.md
Point to the location where ISO is available.
* Update hash reference in publish docs workflow - Updated to the latest `publish-docs` in `orch-ci` * Update publish-docs.yml * Update publish-docs.yml Updated permissions * Update publish-docs.yml Pin to latest SHA * Adds branch pattern 3.0 * Update publish-docs.yml * Change working dir * Update publish-docs.yml
Updated download links for standalone and developer versions
* permission fixes * workflow permission updates * Update .github/workflows/check-spec.yml * Update check-spec.yml --------- Co-authored-by: Anuj Mittal <[email protected]>
9ab9d9a to
82fd945
Compare
cheeyanglee
previously approved these changes
May 9, 2025
Contributor
|
@adimoft can you please rebase your changes for 3.0-dev |
masoodihuzaifa
previously approved these changes
May 13, 2025
masoodihuzaifa
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
masoodihuzaifa
approved these changes
May 13, 2025
cheeyanglee
approved these changes
May 13, 2025
ranjan-dutta
pushed a commit
that referenced
this pull request
May 14, 2025
* Update hash reference in publish docs workflow (#55) - Updated to the latest `publish-docs` in `orch-ci` * Update sha publish docs (#56) * Update hash reference in publish docs workflow - Updated to the latest `publish-docs` in `orch-ci` * Update publish-docs.yml * Update permissions for publish-docs (#57) * Update hash reference in publish docs workflow - Updated to the latest `publish-docs` in `orch-ci` * Update publish-docs.yml * Update publish-docs.yml Updated permissions * Update publish-docs.yml Pin to latest SHA * Adds branch pattern 3.0 * Update publish-docs.yml * Change working dir * updated EMT-D overview documentation (#60) * updated EMT-D overview documentation * fixed typo * Update sb-howto.md (#58) * Update sb-howto.md * Update sb-howto.md * Update sb-howto.md * docs/overview: fix location of iso (#62) Point to the location where ISO is available. * Update pinned sha in publish docs (#64) * Update hash reference in publish docs workflow - Updated to the latest `publish-docs` in `orch-ci` * Update publish-docs.yml * Update publish-docs.yml Updated permissions * Update publish-docs.yml Pin to latest SHA * Adds branch pattern 3.0 * Update publish-docs.yml * Change working dir * Update publish-docs.yml * Update README.md (#74) Updated download links for standalone and developer versions * Zizmor report related fixes (#73) * permission fixes * workflow permission updates * Update .github/workflows/check-spec.yml * Update check-spec.yml --------- Co-authored-by: Anuj Mittal <[email protected]> * Fixed some linter warnings * Fixed small linter issue --------- Signed-off-by: Lee Chee Yang <[email protected]> Co-authored-by: Lee Chee Yang <[email protected]> Co-authored-by: Anuj Mittal <[email protected]> Co-authored-by: Ana Luisa Ponsirenas <[email protected]> Co-authored-by: Mats Agerstam <[email protected]> Co-authored-by: SupriyaPamulpati <[email protected]> Co-authored-by: Ashutosh Kumar <[email protected]> Co-authored-by: Nirmal George <[email protected]>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Merge Checklist
All boxes should be checked before merging the PR
Description
I made this change for fixing a pretty long list of warnings in the linter logs.
Any Newly Introduced Dependencies
No
How Has This Been Tested?
The result can be checked in the Linter logs(compare the logs before/after).