Skip to content

[carlosm] Change data types to charge, mass and monoisotopic_mass in the chemica_data table

Carlos Andres Moreno Velez requested to merge issue-27 into master

Resolves Issue 29

We need to change the data types for the columns charge, mass and monoisotopic_mass in the chemical_data table.

  • This change has security impacts (if so, add them to the description)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README, Confluence etc.)
  • My changes generate no new warnings in the build
  • I have added tests that prove my fix is effective or that my feature works
  • My code does not decrease pylint score

Dependencies

  • This Merge Request is dependent on any other Merge Request or configuration or some ticket; please mention here
  • Includes migration
  • Installs new requirement

Post Review

  • Squash and merge feature_branch to staging
  • Delete branch after merge

Merge request reports