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

Try AutoFormating the code #114

Merged
merged 2 commits into from
Nov 2, 2022
Merged

Try AutoFormating the code #114

merged 2 commits into from
Nov 2, 2022

Conversation

dellis1972
Copy link
Collaborator

No description provided.

@dellis1972 dellis1972 marked this pull request as ready for review November 2, 2022 14:09
@jonathanpeppers jonathanpeppers merged commit b9dfb59 into main Nov 2, 2022
@jonathanpeppers jonathanpeppers deleted the autoformat branch November 2, 2022 14:53
@jonathanpeppers
Copy link
Owner

Do I need to change any settings on this repo? Or can we do a test PR now?

@rolfbjarne
Copy link
Collaborator

@jonathanpeppers actually there's a problem here:

https://github.com/jonathanpeppers/inclusive-code-reviews-ml/blob/main/.github/workflows/autoformat.yml#L18

The workflow is not valid. .github/workflows/autoformat.yml (Line: 18, Col: 13): A sequence was not expected

from here: https://github.com/jonathanpeppers/inclusive-code-reviews-ml/actions/runs/3378235170

projects is a string, so the two solutions must be a single string where each path is separated by a space.

@dellis1972
Copy link
Collaborator Author

@rolfbjarne @jonathanpeppers I'll make the change and push to main

@dellis1972
Copy link
Collaborator Author

ok done

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