Resolve "Replacing Operations in SFG" (!22) · Merge requests · Computer Engineering / B-ASIC - Better ASIC Toolbox · GitLab

Snippets Groups Projects

Merged Jacob Wahlman requested to merge 15-replacing-operations-in-sfg into develop 4 years ago

All threads resolved!

I added some thoughts on how this issue was implemented in #15 (closed), for most part on how we handle replacing several operations. Closes #15 (closed)

Activity

Jacob Wahlman changed milestone to %Period 1 Kanban 4 years ago

changed milestone to %Period 1 Kanban
Jacob Wahlman added Needs review + 1 deleted label 4 years ago

added Needs review + 1 deleted label
Jacob Wahlman assigned to @jacwa448 4 years ago

assigned to @jacwa448
Felix Goding @felgo673 started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by Felix Goding
Last reply by Felix Goding 4 years ago

Jacob Wahlman added 1 commit 4 years ago

added 1 commit

0cf7c24e - removed the ability to multi-replace several operations

Compare with previous version

Felix Goding resolved all threads 4 years ago

resolved all threads

Felix Goding approved this merge request 4 years ago

approved this merge request

Jacob Wahlman added 3 commits 4 years ago

added 3 commits

0cf7c24e...d4ec822b - 2 commits from branch develop
0cc2c7f0 - merged w/ latest develop

Compare with previous version

Rasmus Karlsson approved this merge request 4 years ago

approved this merge request

Jacob Wahlman @jacwa448 · 4 years ago

Author Contributor

@felgo673 Please look at the updated changes.

Felix Goding @felgo673 started a thread on the diff 4 years ago

Resolved 4 years ago by Jacob Wahlman

Felix Goding @felgo673 started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by Jacob Wahlman

Felix Goding @felgo673 started a thread on an old version of the diff 4 years ago

Resolved 4 years ago by Jacob Wahlman

Jacob Wahlman added 3 commits 4 years ago

added 3 commits

37d6eb81 - 1 commit from branch develop
e8a68a4c - merged w/ latest develop
c42bd75f - Fixed some issues w/ the test

Compare with previous version

Jacob Wahlman resolved all threads 4 years ago

resolved all threads

Felix Goding approved this merge request 4 years ago

approved this merge request

Jacob Wahlman @jacwa448 · 4 years ago

Author Contributor

@raska119 Please reapprove for the newest changes.

Rasmus Karlsson approved this merge request 4 years ago

approved this merge request

Jacob Wahlman merged 4 years ago

merged

Jacob Wahlman mentioned in commit 4 years ago

mentioned in commit a1eeb37b

Please register or sign in to reply