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

Simplify Github release CI code #1883

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Simplify Github release CI code #1883

wants to merge 1 commit into from

Conversation

timbze
Copy link
Contributor

@timbze timbze commented Oct 23, 2021

Reason for this

  • actions/create-release is no longer maintained
  • Simplify the yml code

Note

I have not tested it here, but used the same way I did in other personal code

- name: Create release
if: contains(github.ref, 'refs/tags/')
id: create_release
uses: actions/create-release@v1
uses: softprops/action-gh-release@v1
Copy link
Contributor

@tuomas2 tuomas2 Oct 28, 2021

Choose a reason for hiding this comment

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

Suggested change
uses: softprops/action-gh-release@v1
uses: softprops/action-gh-release@6034af24fba4e5a8e975aaa6056554efe4c794d0

Copy link
Contributor

Choose a reason for hiding this comment

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

How can we trust this author? I feel a bit scared always using these third-party modules that have minified js code... almost would like to build code myself from source and review code.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Good question. It was one of the 4 actions recommended from Github's action repo. It was the one with most stars and forks, and looks like it's being constantly maintained.

Copy link
Contributor

Choose a reason for hiding this comment

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

It's pity that Github does not take responsibility of maintaining this kind of useful basic tools...

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm just seeing that the source files are TypeScript. Isn't the minified js file just TypeScript processed? We could quite easily fork and do that process ourselves I think

@timbze timbze closed this Oct 24, 2022
@tuomas2
Copy link
Contributor

tuomas2 commented Oct 24, 2022

I'm not sure if this should be closed even if we can't merge it directly because I'm afraid old code might stop working some time in the future (contains deprecating api).

@tuomas2 tuomas2 reopened this Oct 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Prio 5
Development

Successfully merging this pull request may close these issues.

2 participants