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

[E-idora-create < T464-MAGE] Implement set rel property for create functions - #279

Merged
ind1xa merged 11 commits into
E-idora-add-create-functionsfrom
T464-MAGE-implement-set_rel_property
Aug 11, 2023
Merged

[E-idora-create < T464-MAGE] Implement set rel property for create functions#279
ind1xa merged 11 commits into
E-idora-add-create-functionsfrom
T464-MAGE-implement-set_rel_property

Conversation

@ind1xa

@ind1xa ind1xa commented Jul 31, 2023

Copy link
Copy Markdown
Contributor

Description

Implemented create.set_rel_property and e2e correctness tests for it

Pull request type

  • Bugfix
  • [x ] Algorithm/Module
  • Feature
  • Code style update (formatting, renaming)
  • Refactoring (no functional changes, no api changes)
  • Build related changes
  • Documentation content changes
  • Other (please describe):

######################################

Reviewer checklist (the reviewer checks this part)

Module/Algorithm

######################################

@ind1xa ind1xa added the status: ready PR is ready for review label Jul 31, 2023
@ind1xa
ind1xa requested a review from antoniofilipovic July 31, 2023 13:07
@ind1xa ind1xa self-assigned this Jul 31, 2023
@ind1xa
ind1xa changed the base branch from E-idora-add-create-functions to main August 1, 2023 09:27
@ind1xa
ind1xa changed the base branch from main to E-idora-add-create-functions August 1, 2023 09:28
Comment thread cpp/create_module/algorithm/create.cpp

@antoniofilipovic antoniofilipovic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One discission point

@antoniofilipovic antoniofilipovic added status: discuss it PR commented - needs discussion and removed status: ready PR is ready for review labels Aug 1, 2023
@ind1xa
ind1xa requested a review from antoniofilipovic August 1, 2023 13:06
@ind1xa ind1xa added status: ready PR is ready for review and removed status: discuss it PR commented - needs discussion labels Aug 1, 2023
Comment thread cpp/create_module/algorithm/create.cpp Outdated

@antoniofilipovic antoniofilipovic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Refactor one function according to linked PR

@antoniofilipovic antoniofilipovic added status: change PR reviewed - needs changes and removed status: ready PR is ready for review labels Aug 2, 2023
@ind1xa
ind1xa requested a review from antoniofilipovic August 2, 2023 09:35
@ind1xa ind1xa added status: ready PR is ready for review and removed status: change PR reviewed - needs changes labels Aug 2, 2023
@ind1xa ind1xa changed the title [E-idora < T464-MAGE] Implement set rel property for create functions [E-idora-create < T464-MAGE] Implement set rel property for create functions Aug 2, 2023
@ind1xa ind1xa added status: draft PR is in draft phase and removed status: ready PR is ready for review labels Aug 4, 2023
@ind1xa ind1xa added status: ready PR is ready for review and removed status: draft PR is in draft phase labels Aug 5, 2023
Comment thread cpp/create_module/algorithm/create.cpp Outdated
Comment thread cpp/create_module/algorithm/create.cpp

@antoniofilipovic antoniofilipovic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One structural change

@antoniofilipovic antoniofilipovic added status: change PR reviewed - needs changes and removed status: ready PR is ready for review labels Aug 6, 2023
@ind1xa
ind1xa requested a review from antoniofilipovic August 7, 2023 06:39
@ind1xa ind1xa added status: ready PR is ready for review and removed status: change PR reviewed - needs changes labels Aug 7, 2023

@antoniofilipovic antoniofilipovic left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good to me

@antoniofilipovic antoniofilipovic added status: ship it PR approved and removed status: ready PR is ready for review labels Aug 9, 2023
@antoniofilipovic

Copy link
Copy Markdown
Contributor

Resolve conflicts and wait for checks to pass please

@ind1xa
ind1xa merged commit 184e7ef into E-idora-add-create-functions Aug 11, 2023
@ind1xa
ind1xa deleted the T464-MAGE-implement-set_rel_property branch August 11, 2023 13:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

status: ship it PR approved

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants