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.

Sign up for GitHub

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

Jump to bottom

feat: add content repositories #322

Draft
wants to merge 42 commits into
base: main
Choose a base branch
from
Draft

feat: add content repositories #322

wants to merge 42 commits into from

Conversation

tdstein
Copy link
Collaborator

@tdstein tdstein commented Nov 7, 2024

No description provided.

zackverham and others added 30 commits September 26, 2024 19:22
@zackverham
wip implementation of git repo resource
2de5dd6
@zackverham
WIP: better ergonomics
cb3f83a
@schloerke
Merge branch 'main' into zack-192-git-repo-settings
0dde373
@schloerke
Merge branch 'main' into zack-192-git-repo-settings
cad4808 
* main:
  build: Embrace ruff (#319)
  refactor: inject url path parts instead of endpoints (#315)
@schloerke
Run slow tests last
32e5d8a
@schloerke
Make sure args are never unused
3200268
@schloerke
Add utils file containing drop_none(x)
05ed953
@schloerke
Update hooks.py
22b62ba
@schloerke
Consolidate methods into a single call
02be543 
from earlier conversations with @tdstein , we should only have a single method with a trailiing kwargs so that a single mistype doesn't ruin the typed experience
@schloerke
Fix incorrect default values
3f68756
@schloerke
Only display warnings once for per python session.
087124a 
The situation was really annoying and this is a temp fix
@schloerke
Update resources.py
3168f58
@schloerke
Fix default values to None; Fix some attr access
1fd77ab
@schloerke
Active -> ActiveParams
0ccf961 
Update resources.py
@schloerke
Merge branch 'main' into zack-192-git-repo-settings
224efed 
* main:
  refactor: removes cache layer from active sequence (#320)
@schloerke
Update .gitignore
395a100
@schloerke
Create _api.py
2fda267
@schloerke
Create _json.py
2128318
@schloerke
Small nit updates
f104149
@schloerke
Use new ApiEndpoint class for ContentItemRepository
1c4b79f
@schloerke
Update _api.py
cb7af36
@schloerke
Create test_api_endpoint.py
849ab75
@schloerke
Add note
5e60fce
@schloerke
Add WIP tests for content item repository
19a3249
@schloerke
commit before computer blows up
44b26c7
@schloerke
Make sure content_guid is a kwarg only
d7a25bf
@schloerke
Move creation code to a helper method
1c890e9
@schloerke
Add notes and str / repr methods
7922f93
@schloerke
Do not fetch all data immediately. Instead, use generator
7eeeafc
@schloerke
Update test_content.py
5483a40
@github-actions GitHub Actions
Copy link

github-actions bot commented Nov 7, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1524 1445 95% 0% 🟢

New Files

File Coverage Status
src/posit/connect/_typing_extensions.py 78% 🟢
TOTAL 78% 🟢

Modified Files

File Coverage Status
src/posit/connect/bundles.py 100% 🟢
src/posit/connect/client.py 98% 🟢
src/posit/connect/content.py 96% 🟢
src/posit/connect/hooks.py 100% 🟢
src/posit/connect/jobs.py 100% 🟢
src/posit/connect/oauth/integrations.py 100% 🟢
src/posit/connect/oauth/oauth.py 100% 🟢
src/posit/connect/resources.py 89% 🟢
src/posit/connect/users.py 96% 🟢
src/posit/connect/vanities.py 94% 🟢
TOTAL 97% 🟢

updated for commit: f356a22 by action🐍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tdstein @zackverham @schloerke