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

Update action to publish to pub.dev #189

Closed
wants to merge 1 commit into from

Conversation

MoralCode
Copy link
Collaborator

@MoralCode MoralCode commented Jul 17, 2023

It seems as though the recent merge of #183 should have triggered an automatic build/publish about 10 hours ago but it looks like the workflow failed.

This may help reduce instances of people reporting issues with things that are already fixed (if this is an issue)

It looks like the current action workflow is trying to use saved oauth tokens, when there may be a much easier/better way to do this as documented by flutter themselves.

This PR introduces the changes needed on the repo side for this to work. However, I am unable to make the changes on pub.dev as I don't have access to change the necessary settings

This is mainly a forward-looking fix, so prior versions between 1.8.1 (or whatevers currently on pub.dev) and the latest tagged version will probably need to be manually released.

@Almoullim
Copy link
Owner

Hello @MoralCode, thank you for this PR, the dart-lang/setup-dart action alone only works for dart packages and doesn't work for flutter packages at the moment (dart-lang/setup-dart#68), but I have been able to find a work of using this action for auth and subosito/flutter-action for the publishing. You can have a look here dart.yml

@Almoullim Almoullim closed this Jul 20, 2023
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.

2 participants