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.

Sign up for GitHub

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

Jump to bottom

Ballerina Formatting CLI tool #16003

Merged
merged 15 commits into from
Jun 28, 2019
Merged

Ballerina Formatting CLI tool #16003

merged 15 commits into from
Jun 28, 2019

Conversation

NipunaMarcus
Copy link
Contributor

Purpose

This will add a CLI tool for Ballerina source formatting.

Fixes #15663
Fixes #15373
Fixes #13794

Check List

  • Read the Contributing Guide
  • Required Balo version update
  • Updated Change Log
  • Checked Tooling Support
  • Added necessary tests
    • Unit Tests
    • Spec Conformance Tests
    • Integration Tests
    • Ballerina By Example Tests
  • Increased Test Coverage
  • Added necessary documentation
    • API documentation
    • Module documentation in Module.md files
    • Ballerina By Examples

NipunaMarcus and others added 14 commits June 11, 2019 13:55
@NipunaMarcus
Add formatting CLI tool
6802e5f
@NipunaMarcus
Resolve conflicts
fd4f66e
@NipunaMarcus
Update CHANGELOG.md
ee45e36
@NipunaMarcus @Shan1024
Update license year in pom.xml
f123e9a 
Co-Authored-By: Shan Mahanama <[email protected]>
@NipunaMarcus @Shan1024
Update license year in spotbugs-exclude.xml
328ba22 
Co-Authored-By: Shan Mahanama <[email protected]>
@NipunaMarcus
Add formatting support for type init expression
0062359
@NipunaMarcus
Add requested changes
2e6575f
@NipunaMarcus
Merge branch 'formatting-cli-tool-dev' of https://github.com/NipunaMa…
efafbac 
…rcus/ballerina into formatting-cli-tool-dev
@NipunaMarcus
Add dry run feature and remove confirmation
557894d
@NipunaMarcus
Merge branch 'master' of https://github.com/ballerina-platform/baller…
5e6713b 
…ina-lang into formatting-cli-tool-dev
@NipunaMarcus
Rename FormattingCmd to FormatCmd
0b96d9a
@NipunaMarcus
Resolve conflicts
b0d7e26
@NipunaMarcus
Add requested .help and readme file changes
65d517a
@NipunaMarcus
Resolve conflicts
62fcf0d
@NipunaMarcus NipunaMarcus added Area/Formatting Team/CompilerFETools Semantic API, Formatter, Shell labels Jun 25, 2019
@NipunaMarcus NipunaMarcus changed the title Formatting cli tool dev Ballerina Formatting CLI tool Jun 25, 2019
nadeeshaan
nadeeshaan approved these changes Jun 27, 2019
View reviewed changes
Copy link
Contributor

@nadeeshaan nadeeshaan left a comment

Choose a reason for hiding this comment

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

Reviewed

@NipunaMarcus NipunaMarcus merged commit bbf3446 into ballerina-platform:jballerina Jun 28, 2019
This was referenced Jul 2, 2019
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nadeeshaan nadeeshaan nadeeshaan approved these changes

Assignees
No one assigned
Labels
Area/Formatting Team/CompilerFETools Semantic API, Formatter, Shell
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@NipunaMarcus @nadeeshaan