Add checking workflow and replace create-foxglove-extension package #61
Conversation
This file contains 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
* Change to MIT license Signed-off-by: Carlo van Driesten <[email protected]>
Signed-off-by: Carlo van Driesten <[email protected]>
* Add release * change upload version * create release * add permissions * readme again * Update versions and force semantic version * Allow workflow only on main * Update README.md remove BMW elements * Update package.json * Add node version & remove @foxglove deps * Adjust node version * Add lichtblick/eslint-plugin --------- Signed-off-by: Carlo van Driesten <[email protected]> Co-authored-by: RobinSchniderBMW <[email protected]> Co-authored-by: Carlo van Driesten <[email protected]>
* fix naming "visualizer" -> "converter" Signed-off-by: Carlo van Driesten <[email protected]>
Signed-off-by: Carlo van Driesten <[email protected]>
* add keywords Signed-off-by: Carlo van Driesten <[email protected]>
…tblick-Suite#5) * Integrate ASAM OSI as Dependency for OSI Ground Truth Extension * Point to asam-osi-types github * Update asam-osi-types npm package url * Adjusts to comply with asam-osi-types v3.7.0 upgrade * Logic adjusted to work with v3.7.0 SensorView trace files --------- Co-authored-by: Carlo van Driesten <[email protected]>
* attempt fix workflow Signed-off-by: Carlo van Driesten <[email protected]>
* fix workflow Signed-off-by: Carlo van Driesten <[email protected]>
* dump github var Signed-off-by: Carlo van Driesten <[email protected]>
Signed-off-by: jdsika <[email protected]>
Signed-off-by: Carlo van Driesten <[email protected]>
Signed-off-by: Carlo van Driesten <[email protected]>
Signed-off-by: Acemir Sousa Mendes <[email protected]>
Signed-off-by: Carlo van Driesten <[email protected]>
Signed-off-by: Carlo van Driesten <[email protected]>
) * Automate changelog file update on release using github actions workflows * Add release instructions to README.me * Reset changelog.md * Add pre-commit hook fo Conventional Commits doc(example-data): Create README.md for example data (Lichtblick-Suite#20) * Create README.md for example data * delete outdated example * add OpenPASS as optional simulator --------- Signed-off-by: Carlo van Driesten <[email protected]> ci(release): remove trigger on PRs from workflow (Lichtblick-Suite#21) Ci(fix): release pipeline (Lichtblick-Suite#23) * ci(release): swap direct push on main with a PR * ci(release): add PAT to main push operation Update README.md * add requirement to sign a tag to readme Signed-off-by: Carlo van Driesten <[email protected]> ci(release): attach changelog to release instead of committing to main (Lichtblick-Suite#24) * ci(release): attach changelog to release instead of committing to main * ci(release): add PR creation job fix(ci): release pipeline fix (Lichtblick-Suite#25) * update to node 18 * remove create pull request for now --------- Signed-off-by: Carlo van Driesten <[email protected]> Co-authored-by: Carlo van Driesten <[email protected]> ci: add initial commit history to workflow|bump version to 0.0.2 (Lichtblick-Suite#26) ci(release): update workflow (Lichtblick-Suite#27) Ci release pipeline fix (Lichtblick-Suite#28) * ci(release): use env variables instead of set-output * ci(release): update release Ci release pipeline fix (Lichtblick-Suite#29) * ci(release): use env variables instead of set-output * ci(release): update release * ci(release): add line seperations to initial commit history in changelog Ci release pipeline fix (Lichtblick-Suite#30) * ci(release): use env variables instead of set-output * ci(release): update release * ci(release): fix pipeline workflow ci: adapt pipeline workflow (Lichtblick-Suite#31) ci(release): add initial commit history fetch job to pipeline (Lichtblick-Suite#32) Ci create pr to update changelog file (Lichtblick-Suite#33) * ci(release): add PR creation step to update changelog file * chore(package): bump version to v0.0.3 ci(release): update PR commit message (Lichtblick-Suite#34) ci(release): remove uppercase start from commit message (Lichtblick-Suite#35) ci(release): add PAT to PR creation step (Lichtblick-Suite#36) ci(release): handle staging with create-pull-request action (Lichtblick-Suite#40) ci(release): add commit step to update-changelog branch (Lichtblick-Suite#42) ci(release): add branch pull action before commit (Lichtblick-Suite#43) ci(release): force changes in changelog commit (Lichtblick-Suite#44) ci(release): specifiy pull reconcilation (Lichtblick-Suite#45) ci(release): add changes stashing before and after pull (Lichtblick-Suite#46) Ci create pr to update changelog file (Lichtblick-Suite#47) * ci(release): update workflow * ci(release): add delete branch option ci(release): update workflow (Lichtblick-Suite#49) ci(release): update workflow (Lichtblick-Suite#50) ci(release): update workflow (Lichtblick-Suite#52) ci(release): update workflow (Lichtblick-Suite#53) ci(release): split jobs (Lichtblick-Suite#54) ci(release): preserve state across jobs (Lichtblick-Suite#56) ci(release): update workflow (Lichtblick-Suite#57) ci(release): update workflow (#1) ci(release): update pipeline ci(release): update pipeline
|
@jdsika , Please note that steps: lint and audit are failing due to non compliant code/packages, I bypassed them for now in the workflow, but they should be fixed in the future. Also, it seems that the scancode setup is failing here due to non existent python installation or maybe a different version, it runs in my fork though. Please ignore all the other changes, this happened because I rebased my main branch to reduce the number of commits and have a cleaner history after the tests I performed, If you are okay with it, we can rebase this repo as well after the merge, please let me know. Update: I fixed the python error by adding a python installation step, it seems that the python version is different across runners. |
samikachai
changed the title
|
@jdsika , I added 2 additional commits to replace the create-foxglove-extension package with create-lichtblick-extension, I also updated the node versions in the pipeline from v18 to v20 because the rimraf package was only compatible with v20 ==> v22 of node. This PR should take care of everything regarding this project, please review and merge and I will move forward to other projects. |
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.
No description provided.