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

Update d2198fd60b0e_added_result_run_id.py #143

Merged
merged 2 commits into from
Sep 19, 2023

Conversation

marqueewinq
Copy link
Contributor

@marqueewinq marqueewinq commented Sep 18, 2023

I've made a mistake in #141.

For some reason the generated migration has nullable=False even while schema file has nullable=True, default=None args.

This hotfixes the migrations -- i will also delete the broken version from PyPI.

@marqueewinq marqueewinq self-assigned this Sep 18, 2023
@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #143 (bf05cf8) into master (0c891bf) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #143   +/-   ##
======================================
  Coverage    79.5%   79.5%           
======================================
  Files          11      11           
  Lines         890     890           
  Branches      135     135           
======================================
  Hits          708     708           
  Misses        140     140           
  Partials       42      42           

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@marqueewinq marqueewinq merged commit 761cc73 into master Sep 19, 2023
5 checks passed
@marqueewinq marqueewinq deleted the hotfix-migration-d2198fd60b0e branch September 19, 2023 08:56
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