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

Automate publishing releases on GitHub #249

Merged
4 commits merged into from
Dec 15, 2020
Merged

Automate publishing releases on GitHub #249

4 commits merged into from
Dec 15, 2020

Conversation

ghost
Copy link

@ghost ghost commented Dec 14, 2020

This will automatically publish a release including the latest changelog on GitHub when the release lane is run.

The latest release (2.3.1) was created using this lane.

Running the whole release lane on CI whenever a prepare pr is merged would be the next step in automating the release process.

@ghost ghost self-requested a review December 14, 2020 09:29
@ghost ghost self-assigned this Dec 14, 2020
.gitignore Outdated
Comment on lines 90 to 91
# Vi
.swp
Copy link

Choose a reason for hiding this comment

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

Isn't this file generated anyway? I bet they support vim too.

Copy link

Choose a reason for hiding this comment

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

Alternatively, does it make sense to commit this at all? Would we accept a PR for emacs support in our gitignore?

Maybe this is something users should configure locally, like so? https://stackoverflow.com/a/5724476

Copy link
Author

Choose a reason for hiding this comment

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

Since its mostly me working on this code I quickly setup the gitignore to fit my working environment. User level global gitignores would be a good way of solving this as well. Probably superior even. 👍

If someone regularly commits to the project and wants an emacs gitignore I'd be happy to accept a PR. 😅

Copy link

Choose a reason for hiding this comment

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

Fair enough

version = get_version_number(target: "JOSESwift")

changelog = File.read("../CHANGELOG.md")
changes = changelog[/(- .+\(#.+\)\n)+/]
Copy link

Choose a reason for hiding this comment

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

What is this regex doing? I get you're stripping chars, but why exactly?

Copy link
Author

@ghost ghost Dec 15, 2020

Choose a reason for hiding this comment

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

Filtering out the latest change set.

Blablabla

[1.2.0]

- Do Something (#123)
- Do Something else (#100)

[1.1.0]

- Start something (#99)

Will become:

- Do Something (#123)
- Do Something else (#100)

Which is the change log for the latest version that we want attached to the release.

Copy link

Choose a reason for hiding this comment

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

Might be worth dropping a source comment

Copy link
Author

Choose a reason for hiding this comment

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

Yeah you're right. 👍 Will add this later!

ghost
ghost previously approved these changes Dec 15, 2020
@ghost ghost self-requested a review December 15, 2020 13:41
@ghost ghost dismissed their stale review via cd37b8a December 15, 2020 14:22
@ghost ghost self-requested a review December 15, 2020 14:23
@ghost
Copy link
Author

ghost commented Dec 15, 2020

@nathan-mohemian Updated .gitignore and Fastfile. 👌

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@ghost ghost merged commit e09b939 into master Dec 15, 2020
@ghost ghost deleted the automate-releases branch April 20, 2021 13:59
This pull request was closed.
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.

1 participant