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

ci: add production site deployment #216

Merged
merged 17 commits into from
Jun 11, 2024
Merged

ci: add production site deployment #216

merged 17 commits into from
Jun 11, 2024

Conversation

tdstein
Copy link
Collaborator

@tdstein tdstein commented Jun 10, 2024

No description provided.

Copy link

github-actions bot commented Jun 10, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1018 978 96% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 892056d by action🐍

@nealrichardson
Copy link
Collaborator

What is the resulting production URL? I'm guessing we want this under docs.posit.co, maybe even https://docs.posit.co/posit-sdk-py. That isn't netlify is it?

@tdstein
Copy link
Collaborator Author

tdstein commented Jun 10, 2024

What is the resulting production URL? I'm guessing we want this under docs.posit.co, maybe even https://docs.posit.co/posit-sdk-py. That isn't netlify is it?

It writes to the production Netlify site. Which ops will route to the appropriate docs url.

@tdstein
Copy link
Collaborator Author

tdstein commented Jun 11, 2024

What is the resulting production URL? I'm guessing we want this under docs.posit.co, maybe even https://docs.posit.co/posit-sdk-py. That isn't netlify is it?

It writes to the production Netlify site. Which ops will route to the appropriate docs url.

https://posit-sdk-py.netlify.app

Base automatically changed from tdstein/200 to main June 11, 2024 13:58
@tdstein tdstein merged commit d16252c into main Jun 11, 2024
29 checks passed
@tdstein tdstein deleted the tdstein/209 branch June 11, 2024 14:48
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