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

apply_change_to_project: Raise on project reference without file. #52

Merged

Conversation

byohay
Copy link
Collaborator

@byohay byohay commented Mar 6, 2022

No description provided.

@@ -524,6 +524,12 @@ def add_subproject_reference(root_object, project_reference_change)
find_file(root_object.project, project_reference_change["ProjectRef"],
file_filter: filter_subproject_without_project_references)

unless subproject_reference
raise MergeError, "No file reference was found for project reference with change " \
"#{project_reference_change}. This might mean that the file used to exist in the but " \
Copy link
Member

Choose a reason for hiding this comment

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

where did it use to exist?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

@ashdnazg
Copy link
Member

ashdnazg commented Mar 7, 2022

F&S&M

@byohay byohay force-pushed the feature/raise-project-reference-without-file branch from dd54b88 to f718d7f Compare March 7, 2022 16:10
@byohay
Copy link
Collaborator Author

byohay commented Mar 7, 2022

@ashdnazg fixed (can't merge until you approved)

@byohay byohay merged commit 508276a into Lightricks:main Mar 8, 2022
@byohay byohay deleted the feature/raise-project-reference-without-file branch March 8, 2022 07:19
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.

None yet

2 participants