Skip to content

[ENG-7572] [ENG-7797] [ENG-7859] VRL - Unit tests for GV: M1 & M2#277

Closed
Vlad0n20 wants to merge 7 commits into
CenterForOpenScience:feature/verified-resource-linkingfrom
Vlad0n20:feature/ENG-7859
Closed

[ENG-7572] [ENG-7797] [ENG-7859] VRL - Unit tests for GV: M1 & M2#277
Vlad0n20 wants to merge 7 commits into
CenterForOpenScience:feature/verified-resource-linkingfrom
Vlad0n20:feature/ENG-7859

Conversation

@Vlad0n20

@Vlad0n20 Vlad0n20 commented May 2, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@Vlad0n20
Vlad0n20 force-pushed the feature/ENG-7859 branch from 6d268fd to d84ed75 Compare May 2, 2025 14:47
@Vlad0n20
Vlad0n20 force-pushed the feature/ENG-7859 branch from d84ed75 to 7ef26aa Compare May 2, 2025 15:07
@cslzchen cslzchen changed the title [ENG-7859] Feature/eng 7859 add tests for Dataverse imp as linked addon [ENG-7859] Add tests for Dataverse imp as linked addon May 2, 2025
@cslzchen cslzchen changed the title [ENG-7859] Add tests for Dataverse imp as linked addon [ENG-7572] [ENG-7797] [ENG-7859] M1 & M2 Unit Tests May 12, 2025

@cslzchen cslzchen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Looks good overall.

I see changes in this c7730b6 commit is missing in this PR, do we no longer need it? Please double check.

@Vlad0n20

Copy link
Copy Markdown
Contributor Author

Looks good overall.

I see changes in this c7730b6 commit is missing in this PR, do we no longer need it? Please double check.

Yeah it is no longer necessary

@cslzchen cslzchen left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

🌟

@cslzchen cslzchen changed the title [ENG-7572] [ENG-7797] [ENG-7859] M1 & M2 Unit Tests [ENG-7572] [ENG-7797] [ENG-7859] VRL - Unit tests for GV: M1 & M2 May 21, 2025
@cslzchen

Copy link
Copy Markdown
Collaborator

PR will be merged along with #288

@cslzchen cslzchen closed this May 23, 2025
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.

2 participants