Skip to content

Conversation

tinawang123
Copy link
Contributor

@okurz
Copy link
Member

okurz commented Dec 5, 2024

Please adjust your commit message to be more specific. It reads like you would change all repo files :)

@tinawang123 tinawang123 changed the title Remove .install suffix for repo files Remove .install suffix for repo files on MicroOS-SelfInstall flavor Dec 5, 2024
@Martchus
Copy link
Contributor

Martchus commented Dec 5, 2024

You only changed the PR title. However, I'm wondering whether this change actually is MicroOS-SelfInstall specific. Looking at the code I'm more under the impression that this will change more than that.

The style check is also failing.

@andrii-suse
Copy link
Collaborator

It seemed tricky, so I tried to reproduce and apparently the fix was rather simple, so I submitted #280 , I guess this one can be closed.

@tinawang123
Copy link
Contributor Author

Close this PR, as only impact one job, I will update job settings.

@tinawang123 tinawang123 closed this Dec 6, 2024
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.

4 participants