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

build: Update dependencies to temporarily use the GitHub repo for tm_data_types #2

Merged
merged 5 commits into from
Sep 7, 2024

Conversation

nfelt14
Copy link
Collaborator

@nfelt14 nfelt14 commented Sep 6, 2024

Proposed changes

This updates the dependency for tm_data_types to pull from the GitHub repo temporarily (until tm_data_types is published on PyPI).

Types of changes

What types of changes does your code introduce?
Put an x in the boxes that apply

  • Bug fix (non-breaking change which fixes an issue)
  • CI/CD update (an update to the CI/CD workflows, scripts, and/or configurations)

Checklist

Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

  • I have followed the guidelines in the CONTRIBUTING document
  • I have signed the CLA
  • I have checked to ensure there aren't other open Pull Requests for the same update/change
  • I have created (or updated) an Issue to track the status of this update/change and updated the link in this PR description (see above in the Proposed changes section) using the wording Addresses #<issue_number>
  • I have performed a self-review of my code
  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • Basic linting passes locally with my changes
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have added necessary documentation (if appropriate)
  • I have updated the Changelog with a brief description of my changes

@github-advanced-security
Copy link

This pull request sets up GitHub code scanning for this repository. Once the scans have completed and the checks have passed, the analysis results for this pull request branch will appear on this overview. Once you merge this pull request, the 'Security' tab will show more code scanning analysis results (for example, for the default branch). Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results. For more information about GitHub code scanning, check out the documentation.

Copy link

codecov bot commented Sep 6, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@nfelt14 nfelt14 requested a review from a team as a code owner September 6, 2024 17:00
tests/test_client.py Fixed Show fixed Hide fixed
u625355
u625355 previously approved these changes Sep 6, 2024
u625355
u625355 previously approved these changes Sep 6, 2024
@nfelt14 nfelt14 enabled auto-merge (squash) September 6, 2024 21:36
@nfelt14 nfelt14 requested review from u625355, tekperson and a team September 6, 2024 21:51
@nfelt14 nfelt14 merged commit 73dd3e4 into main Sep 7, 2024
41 of 61 checks passed
@nfelt14 nfelt14 deleted the nfelt14/misc-updates branch September 7, 2024 01:51
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