Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

Rename jballerina command to ballerina #16819

Merged
merged 3 commits into from
Jul 17, 2019
Merged

Rename jballerina command to ballerina #16819

merged 3 commits into from
Jul 17, 2019

Conversation

pahans
Copy link
Contributor

@pahans pahans commented Jul 16, 2019 •
edited
Loading

Purpose

Rename jballerina command to ballerina
Related to #16205

Approach

Samples

Remarks

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

@pahans pahans changed the title rename jballerina command to ballerina Rename jballerina command to ballerina Jul 16, 2019
@kaviththiranga kaviththiranga mentioned this pull request Jul 16, 2019
Closed
@warunalakshitha
Copy link
Contributor

I think we need to change the jballerina command used in integration tests as well

@anupama-pathirage anupama-pathirage added the Team/CompilerFETools Semantic API, Formatter, Shell label Jul 17, 2019
@pahans pahans merged commit f4af8b6 into ballerina-platform:master Jul 17, 2019
@pahans pahans deleted the rename-jballerina-command branch July 17, 2019 06:33
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@hevayo hevayo hevayo approved these changes

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

Successfully merging this pull request may close these issues.
4 participants
@pahans @warunalakshitha @hevayo @anupama-pathirage