Skip to content
This repository was archived by the owner on Jan 19, 2025. It is now read-only.

Conversation

Aclrian
Copy link
Contributor

@Aclrian Aclrian commented Nov 28, 2022

Closes #1120.

Summary of Changes

The migrate command for rename annotation is now working:
If an annotation is mapped via an One/ManyToOneMapping, a new remove annotation would be created for this one apiv2 element.
Otherwise, there are two options:

  1. If the same name is found in the mapping, an unsure @remove annotation would be created for this element.
  2. Else, all apiv2 elements will be annotated with a @todo annotation.

Testing Instructions

run the migrate command or view and run the test_migration.py file

Aclrian and others added 30 commits November 12, 2022 20:21
switching to an other pc - migth not working
<3 to xorg
 from classes, and (global) functions
with included license
and hopefully the error disapperears
@Aclrian Aclrian force-pushed the 1120-migrate-rename-annotations branch 2 times, most recently from 0169073 to d52094c Compare November 30, 2022 19:41
@Aclrian Aclrian force-pushed the 1120-migrate-rename-annotations branch from 8039e02 to e235631 Compare November 30, 2022 20:13
@Aclrian Aclrian marked this pull request as ready for review December 1, 2022 11:48
@Aclrian Aclrian requested a review from lars-reimann as a code owner December 1, 2022 11:48
@lars-reimann
Copy link
Member

If you're done with a feature, feel free to merge it in.

@Aclrian Aclrian merged commit b2cf4bc into main Dec 1, 2022
@Aclrian Aclrian deleted the 1120-migrate-rename-annotations branch December 1, 2022 15:48
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.

Migrate @rename annotations

2 participants