Open
Conversation
Co-authored-by: esmith1729 <esmith1729@users.noreply.github.com>
Co-authored-by: esmith1729 <esmith1729@users.noreply.github.com>
…ge master, and then push to origin
…t to y for Jenkins pipline to run
…mputingGroup/ibex_utils into ticket_7914_update_script
…ed into update_scripts.py Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
…access to BaseTasks' prompt function Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
…asks instance in update_scripts.py Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
…thod Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com> Co-authored-by: zsoltkebel <zsoltkebel@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com> Co-authored-by: zsoltkebel <zsoltkebel@users.noreply.github.com>
Co-authored-by: dtmaclaren <danielmaclaren@users.noreply.github.com>
Comment on lines
+118
to
+121
| f""" | ||
| Attempt an automatic merge of one branch | ||
| {branch_to_merge_from} to another, {branch_to_merge_to} in {dir} | ||
| """ |
Check notice
Code scanning / CodeQL
Statement has no effect Note
Show autofix suggestion
Hide autofix suggestion
Copilot Autofix
AI 9 months ago
To fix the issue, the multi-line f-string should be converted into a proper docstring for the automatic_merge_of_git_remote method. This involves moving the string to immediately follow the def line of the method. Since the string is intended to describe the method's purpose and parameters, it should be formatted as a standard docstring (triple-quoted string) without the f prefix, as no variable interpolation is required.
Suggested changeset
1
installation_and_upgrade/ibex_install_utils/tasks/git_tasks.py
| @@ -117,5 +117,9 @@ | ||
| ) -> None: | ||
| f""" | ||
| Attempt an automatic merge of one branch | ||
| {branch_to_merge_from} to another, {branch_to_merge_to} in {dir} | ||
| """ | ||
| Attempt an automatic merge of one branch to another in a specified directory. | ||
|
|
||
| Args: | ||
| branch_to_merge_from (str): The name of the branch to merge from. | ||
| branch_to_merge_to (str): The name of the branch to merge to. | ||
| dir (str): The directory where the git repository is located. | ||
| """ |
Copilot is powered by AI and may make mistakes. Always verify output.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.