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.

Sign up for GitHub

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

Jump to bottom

Update dependencies #47

Closed
wants to merge 5 commits into from
Closed

Update dependencies #47

wants to merge 5 commits into from
+136 −18

Conversation

soerenmartius
Copy link
Member

@soerenmartius soerenmartius commented May 5, 2021 •
edited
Loading

  • build: update secrets in github actions pipeline
  • build: upgrade pre-commit-hooks to v0.2.3
  • build: upgrade terratest to v0.34.0
  • build: upgrade build-tools to v0.11.0

@soerenmartius soerenmartius requested a review from mariux as a code owner May 5, 2021 17:09
@soerenmartius soerenmartius force-pushed the soerenmartius/updates branch from d3e5ca5 to e2c80df Compare May 5, 2021 17:29
mariux
mariux requested changes May 5, 2021
View reviewed changes
Copy link
Member

@mariux mariux left a comment

Choose a reason for hiding this comment

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

no code was changed just the build system, so it should not be released as a stand alone version. the changes can be released with the next changes

@soerenmartius soerenmartius force-pushed the soerenmartius/updates branch from e2c80df to 681e82d Compare May 18, 2021 23:41
@soerenmartius soerenmartius requested a review from mariux May 18, 2021 23:42
@soerenmartius
Copy link
Member Author

no code was changed just the build system, so it should not be released as a stand alone version. the changes can be released with the next changes

removed the version

@soerenmartius
Copy link
Member Author

closed in favor of #48

@soerenmartius soerenmartius deleted the soerenmartius/updates branch October 30, 2021 08:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mariux mariux Awaiting requested review from mariux mariux is a code owner

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@soerenmartius @mariux