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

[DENA-846] remove release-please, mention new release system in the README #59

Merged
merged 1 commit into from
Jul 31, 2024

Conversation

MarcinGinszt
Copy link
Collaborator

No description provided.

@MarcinGinszt MarcinGinszt requested a review from a team as a code owner July 31, 2024 08:27
Copy link

linear bot commented Jul 31, 2024

@matthewhughes-uw
Copy link
Contributor

Did we record the discussion around why we picked the semver+release please in the first place? If we need to change this I'm wondering if we could've caught it earlier, i.e. if there's something about our previous process that we can learn from for the future

@MarcinGinszt
Copy link
Collaborator Author

MarcinGinszt commented Jul 31, 2024

@matthewhughes-uw Hey, I think our discussions regarding this matter were
here
and here

@matthewhughes-uw
Copy link
Contributor

@matthewhughes-uw Hey, I think our discussions regarding this matter were
here

🤔 that first link just sends me to a file?

@MarcinGinszt
Copy link
Collaborator Author

@matthewhughes-uw my bad, fixed

@MarcinGinszt MarcinGinszt merged commit dc1350e into main Jul 31, 2024
1 check passed
@MarcinGinszt MarcinGinszt deleted the DENA-846 branch July 31, 2024 11:02
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.

3 participants