Skip to content
This repository was archived by the owner on Dec 12, 2023. It is now read-only.

Update AVS version logic#60

Merged
AntiD2ta merged 7 commits into
developfrom
update-avs-version-logic
Aug 29, 2023
Merged

Update AVS version logic#60
AntiD2ta merged 7 commits into
developfrom
update-avs-version-logic

Conversation

@AntiD2ta
Copy link
Copy Markdown
Contributor

Changes:

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

Testing

Requires testing Yes

In case you checked yes, did you write tests? Yes

@AntiD2ta AntiD2ta self-assigned this Aug 18, 2023
@AntiD2ta AntiD2ta requested a review from adriantpaez as a code owner August 18, 2023 16:48
@AntiD2ta AntiD2ta force-pushed the update-avs-version-logic branch from c7f147b to 56c82b7 Compare August 18, 2023 16:48
Copy link
Copy Markdown
Contributor

@adriantpaez adriantpaez left a comment

Choose a reason for hiding this comment

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

  • node_version should be removed from the manifest schema here, as a required field here, and also in this README doc
  • node_version is still used in some testdata files in the package_handler package, for instance here

@AntiD2ta AntiD2ta requested a review from adriantpaez August 24, 2023 13:23
@AntiD2ta AntiD2ta merged commit b3b5107 into develop Aug 29, 2023
@AntiD2ta AntiD2ta deleted the update-avs-version-logic branch August 29, 2023 08:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants