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

#35 Test create and publish a release with automatic semantic versioning #73

Merged
merged 10 commits into from
Aug 1, 2023

Conversation

Zero3141
Copy link
Contributor

@Zero3141 Zero3141 commented Aug 1, 2023

Partly fixes #35

Heiko Kiesel added 5 commits July 31, 2023 13:02
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
@Zero3141 Zero3141 added the bug Something isn't working label Aug 1, 2023
@Zero3141 Zero3141 requested a review from Weltraumschaf August 1, 2023 13:40
@Zero3141 Zero3141 self-assigned this Aug 1, 2023
@Zero3141 Zero3141 changed the title TESTING: Create and publish a release with automatic semantic versioning #35 Test create and publish a release with automatic semantic versioning Aug 1, 2023

- name: Get current version
id: version
run: echo "::set-output name=version::$(./gradlew -q -Prelease.quiet currentVersion)"
Copy link
Member

Choose a reason for hiding this comment

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

What does this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Added a comment before this step to explain its purpose & functionality. Is this sufficient?

Copy link
Member

Choose a reason for hiding this comment

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

That's a good starting point for the reason, but there should also be some info about the concrete syntax (::set-output name=version::). If you have a link which describes this, a link with a hint that it describes the syntax is sufficient.

Heiko Kiesel added 2 commits August 1, 2023 16:24
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
@Zero3141 Zero3141 requested a review from Weltraumschaf August 1, 2023 14:32
Heiko Kiesel added 3 commits August 1, 2023 16:40
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
Signed-off-by: Heiko Kiesel <heiko.kiesel@iteratec.com>
@Weltraumschaf Weltraumschaf merged commit b56fb2e into secureCodeBox:main Aug 1, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Automate publishing of DefectDojo Java Client release to maven central
2 participants