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

Make all artifact arguments optional #90

Open
DM1122 opened this issue May 30, 2022 · 1 comment · Fixed by #98
Open

Make all artifact arguments optional #90

DM1122 opened this issue May 30, 2022 · 1 comment · Fixed by #98
Assignees
@abeerfatima
Labels
architecture A change that involves a rearchitecture of the software good first issue Good for newcomers
Milestone
V1.0

Comments

@DM1122
Copy link
Collaborator

DM1122 commented May 30, 2022

Will facilitate test and trade creation by removing the need to instantiate unused components or values.
@DM1122 DM1122 added good first issue Good for newcomers architecture A change that involves a rearchitecture of the software labels May 30, 2022
@DM1122 DM1122 added this to the V1.0 milestone Jun 4, 2022
@abeerfatima
Copy link
Contributor

Checked all files and they didn't seem to need any changes (?)

abeerfatima added a commit that referenced this issue Jun 11, 2022
@abeerfatima
Add assert statements to catch undefined components in system functio…
fd67b33 
…n calls #95 and Make all artifact arguments optional #90
@DM1122 DM1122 linked a pull request Jun 24, 2022 that will close this issue
Make all artifact arguments optional & Add assert statements to catch undefined components in system functions #98
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@abeerfatima abeerfatima

Labels
architecture A change that involves a rearchitecture of the software good first issue Good for newcomers
Projects
None yet
Milestone
V1.0
2 participants
@DM1122 @abeerfatima