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

feat: add content type helpers #243

Merged
merged 1 commit into from
Jul 29, 2024
Merged

Conversation

tdstein
Copy link
Collaborator

@tdstein tdstein commented Jul 26, 2024

No description provided.

Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1232 1163 94% 0% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
src/posit/connect/content.py 100% 🟢
TOTAL 100% 🟢

updated for commit: ef1cf63 by action🐍

@jonkeane
Copy link
Collaborator

A description would be nice describing why. Also: do we have an issue in the connect (server) repo where we can discuss the possibility of having the server reply back with this split? We implement it here and in connectapi, and that's ok, but it does seem like something that the server should also be able to (self)-describe

@tdstein
Copy link
Collaborator Author

tdstein commented Jul 26, 2024

A description would be nice describing why. Also: do we have an issue in the connect (server) repo where we can discuss the possibility of having the server reply back with this split? We implement it here and in connectapi, and that's ok, but it does seem like something that the server should also be able to (self)-describe

Not that I could find. I created https://github.com/rstudio/connect/issues/27791

@tdstein tdstein requested a review from toph-allen July 29, 2024 15:56
@tdstein tdstein merged commit 046ace7 into main Jul 29, 2024
30 checks passed
@tdstein tdstein deleted the tdstein/content-type-helpers branch July 29, 2024 15:57
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