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

[GSoD 2021] Formatted actions and .vscode files using prettier #142

Closed
wants to merge 12 commits into from
Closed

[GSoD 2021] Formatted actions and .vscode files using prettier #142

wants to merge 12 commits into from

Conversation

iamrajiv
Copy link
Member

@iamrajiv iamrajiv commented Jul 10, 2021

iamrajiv added 2 commits July 11, 2021 01:15
Signed-off-by: Rajiv Ranjan Singh <[email protected]>
Signed-off-by: Rajiv Ranjan Singh <[email protected]>
@iamrajiv iamrajiv changed the title Formatted actions and .vscode files using prettier [GSoD 2021] Formatted actions and .vscode files using prettier Jul 10, 2021
Copy link

@Rohitesh-Kumar-Jain Rohitesh-Kumar-Jain left a comment

Choose a reason for hiding this comment

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

Pass CI tests so that we can continue reviewing

Copy link

@Rohitesh-Kumar-Jain Rohitesh-Kumar-Jain left a comment

Choose a reason for hiding this comment

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

Refactoring looks good to me

@Rohitesh-Kumar-Jain
Copy link

By the way, @iamrajiv, why you used prettier when modern IDEs could have just done in a single click?

on a side note to perform this formatting you just need to press command + shift + L in jet brains IDEs

Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

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

Thanks for enforcing the format to prettier, I believe it's good to have a consistent format across all the json/YAML files.

However, I'd like to keep the following two styles:

  1. Array should be one item per line:
    - branches: [master]
    + branches:
    +   - master
  2. Keep the trailing comma
    -    "typescript"
    +   "typescript",
    ],

Please change your PR to keep the above two styles, and resolve the conflicts before we can continue reviewing it.

Thank you very much!

Signed-off-by: Rajiv Ranjan Singh <[email protected]>
Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

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

Thanks for the update!

Please see my comments and let me know what you think.

.github/workflows/pypi.yml Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
.vscode/settings.json Outdated Show resolved Hide resolved
huan and others added 3 commits September 10, 2021 14:23
Signed-off-by: Rajiv Ranjan Singh <[email protected]>
Signed-off-by: Rajiv Ranjan Singh <[email protected]>
@iamrajiv iamrajiv requested a review from huan September 10, 2021 17:00
Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

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

All the YAML file permissions should be 0644.

100644 → 100755

@iamrajiv iamrajiv requested a review from huan September 11, 2021 11:30
.vscode/settings.json Outdated Show resolved Hide resolved
@iamrajiv iamrajiv requested a review from huan September 16, 2021 16:27
This pull request was closed.
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