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

Updates to reference released dependencies #123

Merged
merged 4 commits into from
Feb 18, 2022

Conversation

Andy-Grigg
Copy link
Collaborator

Closes #47

This PR updates the CI/CD and pyproject.toml files to reference release versions of packages. Also removes download-artifact actions and enables twine upload for releases.

The dependencies are set as follows:

  • openapi-common has a <2.0.0 dependency. i.e. this package will work with any openapi-common that exposes a compatible interface
  • bomanalytics-openapi has a ==1.0.0 dependency, i.e. an exact dependency. Multiple versions of this package could depend on bomanalytics-openapi 1.0.0, but if we ever want to bump bomanalytics-openapi, we will be forced to release a new version of grantami-bomanalytics

@codecov-commenter
Copy link

Codecov Report

Merging #123 (4d81cae) into main (5124e8d) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #123   +/-   ##
=======================================
  Coverage   95.21%   95.21%           
=======================================
  Files          10       10           
  Lines        1358     1358           
=======================================
  Hits         1293     1293           
  Misses         65       65           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5124e8d...4d81cae. Read the comment docs.

Copy link
Collaborator

@da1910 da1910 left a comment

Choose a reason for hiding this comment

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

These look as expected

@Andy-Grigg Andy-Grigg merged commit 8f3a438 into main Feb 18, 2022
@Andy-Grigg Andy-Grigg deleted the maint/use-released-dependencies branch February 18, 2022 17:14
Andy-Grigg pushed a commit that referenced this pull request Jan 11, 2024
Andy-Grigg added a commit that referenced this pull request Jan 16, 2024
* Remove outdated typevars

* Include finishing processes in summary

* Update example BoM

* Create 4-2_BoM_Sustainability.py

* Add example to docs

* Sustainability example changes

* Fix server name

* Tweak working

* Change BoM naming convention

* Add API link and minor formatting tweaks

* Change legend position

* Uncomment integration tests job

* Make sure this class isn't collected by pytest

* Rename with_legislations to with_legislation_ids

* Add trailing space

* Rename external_id on part to external_identity

* Enforce keywords for indicator constructors

* Run black

* Copy check_server from recordlists

* Add variable to disable vm state management

* Fix error message

* Fix typo

* Use absolute masses for secondary processes and use floats

* Draft bom builder example

* Add example to docs

* Fix typing

* Remove some boms

* Improve sustainability integration tests

* Fix error message

* PR feedback

* Apply language suggestions from code review

Co-authored-by: Andy Grigg <[email protected]>

* Add delimiters and fix comments

* Fix GUID field name

* Use method for record references

* Fix test assertions (due to mass changes in BoM)

* Change plotly renderer in examples tests [skip-ci]

* Update BillOfMaterialsEco2301.xsd

* Add category enum

* Update sus. tests to reflect aggregation changes

* Remove outdated todo

* Fix error message

* Fix missing database test

* Add initial docs for BoM Types

- Add rst
- Move init docstrings to class. I hate this, the docstring then says that a class takes a parameter. It does not. The init does.
- Add docstrings for protocols.

* Update files for 2024 R1

* Ignore deserialization

* Exclude the right thing

* Move a docstring and update the other ignorewords...

* Try without tildes

* Pin vale?

* There is no 2.28

* Update index.rst

* Update README.rst

* Clarify aggregation methodology

* Add links to online docs

* Add record identification documentation

* Remove id notes

* Remove "do not instantiate" notes

* Fix message typing

* Update summary results docstrings

* Bump openapi package

* Remove sustainability blank coatings

* Remove sustainability from specifications

* Remove substances from MaterialWithSustainability

* Fix issues with Quantity

- Treat $ as a special case and never handle namespaces
- Handle attributes with no namespace if no default namespace is present

* Add name to transport results

* Update examples.py

* Update unit tests

* Use merge commit ref

* Update tests/test_item_results.py

Co-authored-by: Doug Addy <[email protected]>

* Create BillOfMaterialsEco1711.xsd

* Schemas sub-package

* Remove schemas from examples

* Review comments

* Expect warnings in tests

* Make inputs mandatory

* Naive support for multiple BoM versions

* Add notes about schemas and schemas to docs

* Rename sample bom

* Delete unused test helpers

* Update test_integration_tests.py

* Modify invalid bom to pass client side validation

* Fix fixture name

* Add custom tables for sustainability

* Fix failing integration tests

* Minor feedback

* Fix database key

* Fix vale vocab

* Remove use of regex

* Remove messages assertions

* Update docs

* Remove results classes

* Update tests

* Remove from docs

* Update PartWithSustainability docstring

* Compose record references

* Add part number

* Update tests

* Correct typing

* Update mock responses and associated tests

* Update dependency

* Make mixin into base class

* Use merge commit ref

* Warn on multiple root parts in sust. query result

* Update warning

* Add part and transport references to docs

* Update docs homepage

* Update GitHub org name

* Update MI version requirements

* Add note about product licensing

* Update Process properties names and docs

* Update BoM builder example

* Update example bom

* Remove duplicated BoM

* Rename test BoMs

* Remove duplicated BoM

* Rename paths variables

* Add variables for extra boms

* Use newer version of vale

* Convert to dataclasses

* Update bom types docs

Remove protocols and base class

* Fix bom types issues

* Comment out troublesome properties

NonMIPartReference and BoM annotations

* Add test

* Fix typing and formatting

* Apply suggestions from code review

Co-authored-by: Andy Grigg <[email protected]>

* Remove special handling for ids

* Comment out annotations

* Address more small issues

Remove typical content on materials

* Add comment about BoM "update"

* Add TODOs with issue ID

* Ignore numpydoc warning

* Add missing docstrings

* Address vale issues

* Update accept.txt

* Update accept.txt

* Add test for unexpected args

* Script tweaks

* Update .vale.ini

* Update common identifiers documentation

* Add missing punctuation

* Wording feedback

* Bump actions/checkout from 3 to 4

Bumps [actions/checkout](https://github.com/actions/checkout) from 3 to 4.
- [Release notes](https://github.com/actions/checkout/releases)
- [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md)
- [Commits](actions/checkout@v3...v4)

---
updated-dependencies:
- dependency-name: actions/checkout
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>

* Remove test bom

* Fix doc header

* Add badges to README

* Sustainability docs tweaks

* Fix/ignore reference warnings

* Typo in example

* Clarify env. footprint hierarchy

* Remove TODO

* Updates to bom builder example script

* Update package index

* Add note about item expansion

* Fix error message

* Document aggregation criteria

* Flake8

* Add space

* Language feedback

* Re-ordering

* Add missing mian dependencies

* Poetry lock with update

* Update emails

* Use openapi PR ref

* Drop Python 3.8

* Support Python 3.12

* Fix tox config

* Use merge commit ref

* Update docs dependencies

* Update comment

* Loosen specifications

* Bump pandas from 2.0.3 to 2.1.3

Bumps [pandas](https://github.com/pandas-dev/pandas) from 2.0.3 to 2.1.3.
- [Release notes](https://github.com/pandas-dev/pandas/releases)
- [Commits](pandas-dev/pandas@v2.0.3...v2.1.3)

---
updated-dependencies:
- dependency-name: pandas
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>

* Remove with_stk_records (#123)

* Fix: add repr and example formatting (#122)

* Fix formatting

* Update 4-2_BoM_Sustainability.py

* Add sustainability query repr

* Maint: Update docs deps (#127)

* Feat: Single part in BoM sustainability (#130)

* Expose single part

* Raise error on no parts

* Docs: PM feedback (#132)

* Reduce output

* Add note about relevance of items

* Update accept.txt

* Apply suggestions from code review

Co-authored-by: Andy Grigg <[email protected]>

* Make vale happy

---------

Co-authored-by: Andy Grigg <[email protected]>

* Bump jupytext from 1.15.2 to 1.16.0 (#133)

Bumps [jupytext](https://github.com/mwouts/jupytext) from 1.15.2 to 1.16.0.
- [Release notes](https://github.com/mwouts/jupytext/releases)
- [Changelog](https://github.com/mwouts/jupytext/blob/main/CHANGELOG.md)
- [Commits](mwouts/jupytext@v1.15.2...v1.16.0)

---
updated-dependencies:
- dependency-name: jupytext
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>

* Docs: review feedback (#142)

* RS and Compliance updates

* Sustainability updates

* BoM types updates

* Examples updates

* Fix typo

* Fix typo [skip-ci]

Co-authored-by: Andy Grigg <[email protected]>

---------

Co-authored-by: Andy Grigg <[email protected]>

* Allow null material identity and reference

* Add first draft of general improvements to documentation

* Vale fixes

* Add note to compliance API page

* Move license note to end

* Fix BoM query descriptions

* Fix typo

* Documentation review feedback

* Point at public bomanalytics-openapi

* Fix cname

* Clean up workflows

* Use public bomanalytics-openapi

* Maint/codacy fixes (#388)

* Codacy fixes

* Add dummy variable

* Add metapackage instructions (#390)

* Docs: update product help links (#391)

* Fix broken reference

* Update Product Docs URL

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Ludovic Steinbach <[email protected]>
Co-authored-by: Doug Addy <[email protected]>
Co-authored-by: Doug Addy <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

Switch CI/CD workflows to use pypi for dependencies following openapi-common and bomanalytics-openapi release
3 participants