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.

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

Bugfix/subprocess access to data objects #296

Merged
merged 4 commits into from
Feb 17, 2023

Conversation

essweine
Copy link
Contributor

This PR

  • updates subprocesses so that they inherit data objects from their parent process
  • fixes a bug where call activities were being for subprocesses (this has had no effect up until now because spiff has never distinguished the two)
  • creates a subprocess/call activity in _on_update_hook instead of -on_ready_before and only copies data from the parent at that point

@sonarcloud
Copy link

sonarcloud bot commented Feb 17, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

97.5% 97.5% Coverage
0.0% 0.0% Duplication

@essweine essweine force-pushed the bugfix/subprocess-access-to-data-objects branch from 950194a to 6d2a203 Compare February 17, 2023 17:51
@danfunk danfunk merged commit b439f69 into main Feb 17, 2023
@danfunk danfunk deleted the bugfix/subprocess-access-to-data-objects branch February 17, 2023 17:59
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.

3 participants