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

First GPOs guide draft #81

Merged
merged 8 commits into from
May 22, 2024
Merged

First GPOs guide draft #81

merged 8 commits into from
May 22, 2024

Conversation

GabrielaReyna
Copy link
Contributor

@GabrielaReyna GabrielaReyna commented May 7, 2024

Copy link

cloudflare-pages bot commented May 7, 2024

Deploying labs-leaningtech-com with  Cloudflare Pages  Cloudflare Pages

Latest commit: 94d45ec
Status: ✅  Deploy successful!
Preview URL: https://2b177a82.labs-leaningtech-com.pages.dev
Branch Preview URL: https://jnlp-runner-gpos.labs-leaningtech-com.pages.dev

View logs

@bates64
Copy link
Member

bates64 commented May 8, 2024

Tests are failing because of #80

@GabrielaReyna
Copy link
Contributor Author

I'm getting broken link errors, it is trying to map to the older cheerpj-jnlprunner url. Any ideas on how to fix this?

@bates64
Copy link
Member

bates64 commented May 21, 2024

I'm getting broken link errors, it is trying to map to the older cheerpj-jnlprunner url. Any ideas on how to fix this?

Yep, this is a known issue #80

@bates64
Copy link
Member

bates64 commented May 22, 2024

Commit 94d45ec could have "fix #80" in the commit message so GitHub knows to close the issue when it is merged. See https://docs.github.com/en/issues/tracking-your-work-with-issues/closing-an-issue. I'll edit the PR description instead in this instance

@bates64
Copy link
Member

bates64 commented May 22, 2024

Do we prefer that Stefano reviews this? If my changes were resolved I'm happy

@bates64 bates64 closed this May 22, 2024
@bates64 bates64 reopened this May 22, 2024
@bates64 bates64 requested review from sderossi and removed request for bates64 May 22, 2024 13:08
@bates64 bates64 marked this pull request as ready for review May 22, 2024 13:08
Copy link
Member

@bates64 bates64 left a comment

Choose a reason for hiding this comment

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

LGTM, assigning @sderossi for second content review :shipit:

@bates64 bates64 merged commit 16cbc6c into main May 22, 2024
16 checks passed
@bates64 bates64 deleted the jnlp-runner-gpos branch May 22, 2024 15:47
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.

Change /cheerpj-jnlprunner/ links to /cheerpj-jnlp-runner/
2 participants