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 nuget org push #2079

Merged
merged 21 commits into from
Feb 2, 2023
Merged

Automate nuget org push #2079

merged 21 commits into from
Feb 2, 2023

Conversation

stesee
Copy link
Contributor

@stesee stesee commented Feb 1, 2023

Updated api keys

There are two steps that I wont be able to prepare... btw. not able without authorization at nuget.org to change your package.

  • Obtain an api key from nuget.org and add it using the key "NUGET_API_KEY" to your secrets to enable pushing built packages to nuget.org. That will get triggerd on pushing to "release-*" branches.
  • Obtain an api key from https://int.nugettest.org/ and add it using the key "NUGET_INT_API_KEY" to your secrets to enable pushing built packages to int.nugettest.org. That will get triggered on pushing to "master" branch and will run online similar checks on the package like need to be satisfied publishing using nuget.org.

@stesee
Copy link
Contributor Author

stesee commented Feb 1, 2023

Is it possible that the tests are a bit flaky? I had a quick look into the failing test running on appveyor and is saw at least PuppeteerSharp.Tests.TargetTests.BrowserWaitForTargetTests.ShouldWaitForATarget failing some times running on github actions too.

.github/workflows/dotnet.yml Outdated Show resolved Hide resolved
@@ -48,3 +49,43 @@ jobs:
export DISPLAY=:1.5
cd lib/PuppeteerSharp.Tests
dotnet test -f net6.0 -s test.runsettings -c Debug --logger "trx;LogFileName=TestResults.xml"

deployRelease:
if: startsWith( github.ref, 'refs/heads/release-' )
Copy link
Member

Choose a reason for hiding this comment

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

Does this mean the tags need to begin with release- or is this a Github convention?
We could just listen to git tags.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Not sure. I used refs/heads/ always to filter branches and used marvinpinto/action-automatic-releases to add tags. I can look that up and report back later.

Are you using tags added by hand to mark releases?

Copy link
Member

Choose a reason for hiding this comment

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

I use the Release feature from github that creates a tag.

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 think you would like to have something that is not filtered to any tag or branch and triggered by you without any automated trigger. Have I got it right? Then it would make sense to pull it out into a new workflow.

Copy link
Member

Choose a reason for hiding this comment

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

Ideally. If we can get a new version when I hit "publish release' and a tag is created, that would be awesome.

.github/workflows/dotnet.yml Outdated Show resolved Hide resolved
.github/workflows/dotnet.yml Outdated Show resolved Hide resolved
@kblok
Copy link
Member

kblok commented Feb 1, 2023

Yes, tests can be quite flaky :(

Copy link
Member

@kblok kblok left a comment

Choose a reason for hiding this comment

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

I like it! we are close!

@kblok
Copy link
Member

kblok commented Feb 2, 2023

@stesee just to confirm. This is ready to go right?

@stesee
Copy link
Contributor Author

stesee commented Feb 2, 2023

Found some more issues while testing - give the new workflow PushToIntNugetOrg a try 🤖.

@kblok kblok merged commit cab8f8e into hardkoded:master Feb 2, 2023
@kblok
Copy link
Member

kblok commented Feb 2, 2023

@stesee ops https://github.com/hardkoded/puppeteer-sharp/actions/runs/4078318164/jobs/7028385175

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