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

Migrate to GitHub actions #31

Merged
merged 3 commits into from
Aug 29, 2023
Merged

Migrate to GitHub actions #31

merged 3 commits into from
Aug 29, 2023

Conversation

Alex0jk
Copy link
Collaborator

@Alex0jk Alex0jk commented Mar 31, 2023

Add basic setup for GitHub actions : release.yml and draft.yml

This is for #30

@Alex0jk Alex0jk self-assigned this Mar 31, 2023
@Alex0jk Alex0jk requested a review from sestrella April 21, 2023 17:15
Copy link
Member

@sestrella sestrella left a comment

Choose a reason for hiding this comment

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

@Alex0jk nice job so far! I would just suggest updating the status badges on the README file and also adding a section talking about the release process, if I understand this workflow correctly a maintainer would need to create a GH release before publishing a new version to Hackage. Am I right?

@Alex0jk Alex0jk force-pushed the gh-actions-setup branch from 474a7ba to 5805f4d Compare July 18, 2023 13:30
@Alex0jk Alex0jk force-pushed the gh-actions-setup branch from 5805f4d to 7c1d2c4 Compare July 18, 2023 13:39
Copy link
Member

@sestrella sestrella left a comment

Choose a reason for hiding this comment

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

@Alex0jk looks good, please let me know if you need some help setting up the secrets

Copy link

@CristhianMotoche CristhianMotoche left a comment

Choose a reason for hiding this comment

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

@Alex0jk Looks good. Although, we don't have an executable on atomic-write so we don't need many of the steps that you defined in the draft.yml file. Also, can you give a look at the README.md? It seems the diagram is not rendering correctly. Thanks!

Comment on lines 14 to 22
```mermaid
graph LR;
event[GH Event]-->|on push|Build;
event-->|tag created|Draft;
Draft-->|create draft release|End;
event-->|release published|Release;
Release-->|upload artifacts to Hackage (release candidate)|End;
Build-->End;
```

Choose a reason for hiding this comment

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

@Alex0jk It seems this diagram has some kind of error. It cannot be rendered. Can you double check that please?

Comment on lines 15 to 24
- uses: addnab/docker-run-action@v3
with:
image: utdemir/ghc-musl:v24-ghc922
options: -v ${{ github.workspace }}:/mnt
run: |
cd /mnt
sh build.static.sh

- name: Change owner before compression
run: sudo chown $USER:$USER atomic-write

Choose a reason for hiding this comment

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

@Alex0jk atomic-write doesn't provide an executable. It's just a library. We don't need any step related to uploading a binary to a GH release.

Copy link

@CristhianMotoche CristhianMotoche left a comment

Choose a reason for hiding this comment

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

LGTM!

@Alex0jk Alex0jk merged commit f71cc5f into master Aug 29, 2023
@Alex0jk Alex0jk deleted the gh-actions-setup branch August 29, 2023 13:21
Copy link

🎉 This PR is included in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Copy link

🎉 This PR is included in version 2.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

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

Successfully merging this pull request may close these issues.

3 participants