Skip to content
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

runtools enhancements & build script for analyzer container #162

Merged
merged 7 commits into from
Jun 26, 2023
Merged

runtools enhancements & build script for analyzer container #162

merged 7 commits into from
Jun 26, 2023

Conversation

AOrps
Copy link
Contributor

@AOrps AOrps commented Jun 14, 2023

  • Changed how argument parsing works
    • included a usage function (will flesh it out more options later)
    • will pick the first instance of a pkg:<pkg_manager>/<pkg>@<version_num> as opposed to the first parameter
  • Changed blue to bright cyan on terminal colors (went from being invisible on dark terminals to readable)
  • dynamic version resolution:
  • added a build script that (tries to closely) mirror the build.ps1 functionality for the analyzer container
  • updated toolshed version number

Signed-off-by: AOrps <aorbeandrews@gmail.com>
Signed-off-by: AOrps <aorbeandrews@gmail.com>
Signed-off-by: AOrps <aorbeandrews@gmail.com>
Signed-off-by: AOrps <aorbeandrews@gmail.com>
Signed-off-by: AOrps <aorbeandrews@gmail.com>
Signed-off-by: AOrps <aorbeandrews@gmail.com>
@AOrps AOrps changed the title Aorps/runtools script enchancements runtools enhancements Jun 14, 2023
@AOrps
Copy link
Contributor Author

AOrps commented Jun 14, 2023

Needed to force-push to signoff on all commits to be compliant with DCO

@AOrps AOrps changed the title runtools enhancements runtools enhancements & build script for analyzer container Jun 14, 2023
Signed-off-by: AOrps <aorbeandrews@gmail.com>
@Cyber-JiuJiteria Cyber-JiuJiteria merged commit 29e2671 into ossf:main Jun 26, 2023
Copy link
Contributor

@Cyber-JiuJiteria Cyber-JiuJiteria left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

compelted review. merging

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants