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.

Sign up for GitHub

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

Jump to bottom

Feature/olm skip range add to csv annotations #251

Merged
merged 3 commits into from
Mar 17, 2022
Merged

Feature/olm skip range add to csv annotations #251

merged 3 commits into from
Mar 17, 2022

Conversation

barakda
Copy link
Contributor

@barakda barakda commented Mar 17, 2022

No description provided.

oriyarde
oriyarde previously approved these changes Mar 17, 2022
View reviewed changes
Copy link
Contributor

@oriyarde oriyarde left a comment

Choose a reason for hiding this comment

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

LGTM

matancarmeli7
matancarmeli7 previously approved these changes Mar 17, 2022
View reviewed changes
Copy link
Contributor

@matancarmeli7 matancarmeli7 left a comment

Choose a reason for hiding this comment

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

LGTM

tyichye19
tyichye19 previously approved these changes Mar 17, 2022
View reviewed changes
Copy link
Contributor

@tyichye19 tyichye19 left a comment

Choose a reason for hiding this comment

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

LGTM

@barakda barakda dismissed stale reviews from tyichye19, matancarmeli7, and oriyarde via 8cae7dd March 17, 2022 12:09
oriyarde
oriyarde approved these changes Mar 17, 2022
View reviewed changes
Copy link
Contributor

@oriyarde oriyarde left a comment

Choose a reason for hiding this comment

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

LGTM

@barakda barakda merged commit 7a74e93 into develop Mar 17, 2022
@barakda barakda deleted the feature/olm_skipRange_add_to_csv_annotations branch March 17, 2022 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@matancarmeli7 matancarmeli7 matancarmeli7 approved these changes

@oriyarde oriyarde oriyarde approved these changes

@tyichye19 tyichye19 tyichye19 left review comments

@annaparienti annaparienti Awaiting requested review from annaparienti

@ArbelNathan ArbelNathan Awaiting requested review from ArbelNathan

@roysahar roysahar Awaiting requested review from roysahar

@erantzabari erantzabari Awaiting requested review from erantzabari

@samishekh samishekh Awaiting requested review from samishekh

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@barakda @matancarmeli7 @oriyarde @tyichye19 @ghnvmeof