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

Draft new release to fix GH Action on Marketplace #95

Merged
merged 3 commits into from
May 10, 2022

Conversation

tooomm
Copy link
Contributor

@tooomm tooomm commented May 8, 2022

Hi @filiph. I took a look and tried to help with making your tool accessible as predefined action for users to easily include in their workflows. This should probably fix the issues described in #93.

After recent updates from @errnesto tried to fix the action, there was still a release missing which included these changes/fixes.
The available version for easy consumption at the GitHub Actions Marketplace is pointing at the last available release still (2.0.20 @ Marketplace), which does not contain those fixes yet. Hence a new, updated release is needed:
Untitled

Changes:

  • Updated changelog
  • Incremented version number

If you want to change the wording or not include every PR in the changelog let me know or update my branch.

After merging this in, you would only need to tag a new release at the most recent commit on master and your release workflow should trigger and deploy a new version.
The GitHub Actions Marketplace should then pick it up as well and it should finally work. 👍

Feel free to squash my commits on merge.

Copy link
Owner

@filiph filiph left a comment

Choose a reason for hiding this comment

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

This is great, thank you!

@filiph filiph merged commit 0300a9a into filiph:master May 10, 2022
@filiph
Copy link
Owner

filiph commented May 10, 2022

(Please do let me know if there's anything I need to do in GH Actions web administration.)

@tooomm tooomm deleted the patch-1 branch May 10, 2022 13:54
@tooomm
Copy link
Contributor Author

tooomm commented May 10, 2022

@filiph That's awesome, thanks for taking a look and merging this. I hope it finally resolves things for GH Action users.


(Please do let me know if there's anything I need to do in GH Actions web administration.)

You need to create a new release here on GitHub. It looks like you have a GitHub Actions workflow for releases setup to automate this and deploy the assets to it.
In order to trigger it, it should be sufficient to create a new tag with the new version number targeting your master branch/the last commit you just merged. Same as you did for the former releases.

I hope that helps?

@filiph
Copy link
Owner

filiph commented May 10, 2022

Ah, yes. Done that now. Let's see if it works. Thanks again!

@tooomm
Copy link
Contributor Author

tooomm commented May 10, 2022

Well, looks like your release workflow is running into an issue and failing at the grind pkg-github-all command with

pkg-compile-native
failed: Your SDK doesn't have dart2aot or dart2native. This probably means that you're using a 32-bit SDK, which doesn't support native compilation.

I'm not familiar with dart or grind however.


Maybe Guillaume (@Mogztter) can help here. He added the workflow last year.

@filiph
Copy link
Owner

filiph commented May 10, 2022

Ok thanks, I asked about the error over at cli_pkg: google/dart_cli_pkg#101

@ggrossetie
Copy link
Contributor

I've never encountered this error, I searched for an existing issue on dart_cli_pkg but couldn't find one.
I think this is the right call to open an issue at dart_cli_pkg, hopefully they will know how to fix it 🤞

@filiph
Copy link
Owner

filiph commented May 11, 2022

My bad, the project used an old version of cli_pkg. Upgraded that now, let's see what happens.

@filiph
Copy link
Owner

filiph commented May 11, 2022

It's alive! https://github.com/filiph/linkcheck/releases/tag/2.0.23

Can either of you please check that this does indeed work as expected? I'm not using Marketplace myself, so I wouldn't know where to start.

Thank you so much!

@tooomm
Copy link
Contributor Author

tooomm commented May 11, 2022

That are great news! Happy to hear the issue was solved that quickly.

It looks like the Marketplace did not pick up the new version yet.
But it turns out that is simply not a thing and it requires manual updates for now, even though that is not documented.

Solution from the GitHub Community:

  • Go to the repo’s releases page and click on edit on the release I want to publish to the marketplace.
  • Click on update release.
    Note that I do not change anything in the release and that the “Publish this Action to the GitHub Marketplace” is already checked and all checks are green.

Actually, you could also remove the Publish this Action to the GitHub Marketplace checkmark for the 2.0.20 release as it is broken. Not sure about 2.0.15+1.
And the list of changes for 2.0.22 is missing linebreaks. There is an open issue about that on the dart_cli_pkg repo, but it looks like it's intended and the changelog file would need other formatting.


Note: I also found these hints on releasing for the future! https://github.com/filiph/linkcheck#releasing-a-new-version

@filiph
Copy link
Owner

filiph commented May 11, 2022

Thanks @tooomm for guiding me through this. I just pressed the "update release" button. Let's see if that does the trick.

@tooomm
Copy link
Contributor Author

tooomm commented May 11, 2022

I just pressed the "update release" button. Let's see if that does the trick.

That did it... 😄


Can either of you please check that this does indeed work as expected? I'm not using Marketplace myself, so I wouldn't know where to start.

It works now! People can now easily use your tool when adding this to their action workflows:

- name: Check links
  uses: filiph/linkcheck@2.0.23
  with:
    arguments:

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