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

GHA: release management for the firebase C++ SDK #1

Closed
wants to merge 1 commit into from

Conversation

compnerd
Copy link
Collaborator

@compnerd compnerd commented Aug 7, 2023

Add some rules to release the content from the build to preserve the content. Furthermore, begin packaging and deploying a nuget package enabling the future consumption within the Arc build.

Add some rules to release the content from the build to preserve the
content.  Furthermore, begin packaging and deploying a nuget package
enabling the future consumption within the Arc build.
@@ -52,3 +52,37 @@ jobs:
with:
name: firebase-windows-amd64
path: ${{ github.workspace }}/BuildRoot/Library/firebase

- run: |
echo build_tag=$(date +%Y%m%d.$(date +%-H/6 | bc)) >> ${GITHUB_ENV}

Choose a reason for hiding this comment

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

lol what's up w/ the inner $()?
(why is quarter-day a meaningful unit here?)

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Quarter day is a meaningful unit only in that it is copy-pasta. We really just need a counter. The toolchain builds build occur every 6 hours, so we get quarter days.

Choose a reason for hiding this comment

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

lol IMO just use full hours.minutes.seconds and avoid potential future confusion. But that's just me.

- name: publish nuget
env:
GITHUB_TOKEN: ${{ secrets.NUGET_TOKEN }}
GITHUB_USERNAME: thebrowsercompany-bot2

Choose a reason for hiding this comment

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

I guess this isn't something we're hoping to upstream?

Copy link
Collaborator Author

@compnerd compnerd Aug 7, 2023

Choose a reason for hiding this comment

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

Nope, I see no reason we would ever upstream the nuget package. If anything, we might consider switching to the upstream package if we could.

run: |
nuget sources list | findstr "%NUGET_SOURCE_NAME%"
if errorlevel 0 ( nuget sources remove -name %NUGET_SOURCE_NAME% )
nuget sources Add -Name %NUGET_SOURCE_NAME% -Source %NUGET_PUBLISH_URL% -username %GITHUB_USERNAME% -password %GITHUB_TOKEN% -StorePasswordInClearText

Choose a reason for hiding this comment

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

IDK what "StorePasswordInClearText" means but it sounds...exciting?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Exciting is one word for it. This is copy-pasta from thebrowsercompany/swiftwinrt's rules.

@@ -0,0 +1,41 @@
<?xml version="1.0"?>

Choose a reason for hiding this comment

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

I know absolutely nothing about nuget/nuspec so might be nice to get a review from someone who does.

@compnerd compnerd closed this Dec 7, 2023
@compnerd compnerd deleted the release branch December 7, 2023 20:19
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