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

rename Python package to bitwarden_sdk #399

Merged

Conversation

tangowithfoxtrot
Copy link
Contributor

Type of change

  • Bug fix
  • New feature development
  • Tech debt (refactoring, code cleanup, dependency upgrades, etc)
  • Build/deploy pipeline (DevOps)
  • Other

Objective

Rename the Python package. This should be merged before #369.

Code changes

  • languages/python/setup.py: Rename package
  • languages/python/bitwarden_sdk: Rename directory to match package name
  • .gitignore: Changed path to schemas.py
  • .github/workflows/generate_schemas.yml: Changed path to schemas.py

Before you submit

  • Please add unit tests where it makes sense to do so (encouraged but not required)

@tangowithfoxtrot tangowithfoxtrot requested a review from a team December 5, 2023 11:35
@tangowithfoxtrot tangowithfoxtrot marked this pull request as ready for review December 5, 2023 11:41
@tangowithfoxtrot tangowithfoxtrot requested a review from a team as a code owner December 5, 2023 11:41
@bitwarden-bot
Copy link

Logo
Checkmarx One – Scan Summary & Details8cf68c86-4e7f-4a7c-8554-86b763e101d7

No New Or Fixed Issues Found

@Hinton
Copy link
Member

Hinton commented Dec 5, 2023

The CI seems to be failing.

@tangowithfoxtrot
Copy link
Contributor Author

It's failing because the generate_schemas workflow is running from master, which looks for the BitwardenClient dir, which was renamed. The workflow will succeed once my updates are merged into it.

Copy link
Contributor

@cd-bitwarden cd-bitwarden left a comment

Choose a reason for hiding this comment

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

LGTM!

@tangowithfoxtrot tangowithfoxtrot merged commit 7ececc5 into master Dec 6, 2023
36 of 37 checks passed
@tangowithfoxtrot tangowithfoxtrot deleted the sm/SM-1025/Rename-Python-package-to-bitwarden_sdk branch December 6, 2023 18:03
@tangowithfoxtrot tangowithfoxtrot mentioned this pull request Dec 6, 2023
5 tasks
tangowithfoxtrot added a commit that referenced this pull request Dec 6, 2023
## Type of change

- [x] Bug fix
- [ ] New feature development
- [ ] Tech debt (refactoring, code cleanup, dependency upgrades, etc)
- [ ] Build/deploy pipeline (DevOps)
- [ ] Other

## Objective

Fix broken schema generation workflow from #399.

## Code changes

- **./support/scripts/schemas.ts:** Update the Python schemas dir

## Before you submit

- Please add **unit tests** where it makes sense to do so (encouraged
but not required)
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.

5 participants