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

refactor!: Rename File system options from PRELOAD/ON_DEMAND to COPIED/VIRTUAL #40

Merged
merged 1 commit into from
Oct 10, 2023

Conversation

pH-amz
Copy link
Contributor

@pH-amz pH-amz commented Sep 25, 2023

What was the problem/requirement? (What/Why)

AssetLoadingMethod variable and its options were renamed in deadline-cloud library. This change is a downstream effect of that.

What was the solution? (How)

Rename AssetLoadingMethod and its options to new names

What is the impact of this change?

This is a downstream change due to changes in deadline-cloud

How was this change tested?

By running unit tests

Was this change documented?

N/A

Is this a breaking change?

Yes

@pH-amz pH-amz requested a review from a team as a code owner September 25, 2023 17:27
@pH-amz pH-amz force-pushed the pH-amz/filesystem-options-rename branch from 89823de to 31cb93c Compare September 25, 2023 17:27
@pH-amz pH-amz force-pushed the pH-amz/filesystem-options-rename branch 3 times, most recently from 3b4c3a9 to 595e027 Compare October 10, 2023 17:51
@agerveshi
Copy link

I would say this is a breaking change, in that it shouldn't go in with a patch release.

…D/VIRTUAL

BREAKING CHANGES:
   The changes here are a result of breaking changes in the deadline-cloud library.
    - AssetLoadingMethod class is renamed to JobAttachmentsFileSystem
    - PRELOAD, ON_DEMAND enum string values for AssetLoadingMethod are now called COPIED, VIRTUAL

Signed-off-by: Phanindra Bhagavatula <[email protected]>
@pH-amz pH-amz force-pushed the pH-amz/filesystem-options-rename branch from 595e027 to 6faadd6 Compare October 10, 2023 18:55
@pH-amz
Copy link
Contributor Author

pH-amz commented Oct 10, 2023

I would say this is a breaking change, in that it shouldn't go in with a patch release.

I have updated the commit message to reflect the change.

@pH-amz pH-amz changed the title refactor: Rename File system options from PRELOAD/ON_DEMAND to COPIED/VIRTUAL refactor!: Rename File system options from PRELOAD/ON_DEMAND to COPIED/VIRTUAL Oct 10, 2023
@pH-amz pH-amz merged commit fd71650 into mainline Oct 10, 2023
8 checks passed
@pH-amz pH-amz deleted the pH-amz/filesystem-options-rename branch October 10, 2023 19:39
jusiskin pushed a commit to jusiskin/deadline-cloud-worker-agent that referenced this pull request Sep 4, 2024
…dline#40)

Updates the requirements on [mypy](https://github.com/python/mypy) to permit the latest version.
- [Changelog](https://github.com/python/mypy/blob/master/CHANGELOG.md)
- [Commits](python/mypy@v1.6.0...v1.7.0)

---
updated-dependencies:
- dependency-name: mypy
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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