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

Release 2.0 #138

Merged
merged 20 commits into from
Oct 10, 2023
Merged

Release 2.0 #138

merged 20 commits into from
Oct 10, 2023

Conversation

pszulczewski
Copy link
Contributor

develop to main for the release

nautics889 and others added 20 commits April 16, 2023 04:21
Add method `custom_field_choices()` in `App` class.
Rename method `custom_choices()` to `custom_fields()`.
Update docstings for both methods according to returning data.
Add test case `AppCustomFieldChoicesTestCase` for
`custom_field_choices()` method.
Rename test case for `custom_fields()` method.
Add using fixtures for mentioned test cases.
Add missing JSON files with fixtures for tests for getting custom
fields.
Fix test cases `AppCustomFieldsTestCase` and
`AppCustomFieldChoicesTestCase` for python 3.7 stable.
Update logic for checking passed arguments of mock's `call_args`.
* (docs) update docstrings for `custom_fields()` and
  `custom_field_choices()` methods
* (refactor): use f-strings instead of `.format()` in
  `custom_fields()` and `custom_field_choices()`
* (tests): update naming in tests
* (tests): use `return_value` instead of `side_effect` for mocks
* (fix): restore original method `custom_choices()` for application
  class to provide a compatibility with existing client code
* (enhance): add logging a deprecation warning
* (refactor): call `custom_fields()` in `custom_choices()` since they
  represent essentially identical requests to Nautobot
…130)

Add version constraint to divide into two release trains 1.x and 2.x
* Release 2.0.0

---------

Co-authored-by: Jan Snasel <[email protected]>
Co-authored-by: Joe Wesch <[email protected]>
@pszulczewski pszulczewski requested a review from jvanderaa October 5, 2023 16:38
@pszulczewski
Copy link
Contributor Author

@jvanderaa @joewesch can you approve this PR. That's the last step to cut out 2.0 release.

@pszulczewski pszulczewski requested a review from joewesch October 9, 2023 09:23
@pszulczewski pszulczewski merged commit 365c287 into main Oct 10, 2023
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