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

Initialize subprojects and command-line interface #3

Merged
merged 11 commits into from
Mar 23, 2021

Conversation

petertrr
Copy link
Member

@petertrr petertrr commented Mar 18, 2021

What's done:

  • Added subprojects
  • Added CLI runner
  • Added core classes
  • Added versions and talaiot gradle plugins

Closes #2

### What's done:
* WIP
### What's done:
* Removed accidentally committed files
* Bumped gradle version
### What's done:
* WIP
# Conflicts:
#	build.gradle.kts
#	buildSrc/build.gradle.kts
### What's done:
* Fixes
### What's done:
* Fixes
@petertrr petertrr changed the title Feature/base core cli Initialize command-line interface Mar 18, 2021
@petertrr petertrr changed the title Initialize command-line interface Initialize subprojects and command-line interface Mar 18, 2021
### What's done:
* Code style
### What's done:
* Added talaiot and versions plugin
* Changed version of junit dependency
### What's done:
* Added opt-in for okio FS API
* @param args CLI args
* @return an instance of [SaveConfig]
*/
fun createConfigFromArgs(args: Array<String>): SaveConfig {
Copy link
Member

Choose a reason for hiding this comment

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

may be will add some argument for a baseline? (while I haven't yet forgotten about that)
We should have a baseline functionality

Copy link
Member Author

Choose a reason for hiding this comment

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

Added this and addressed other comments, please take a look

Copy link
Member

@orchestr7 orchestr7 left a comment

Choose a reason for hiding this comment

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

please see comments and we will merge it as a basic template concept

### What's done:
* Fixes
@petertrr petertrr requested a review from orchestr7 March 22, 2021 11:41
@petertrr petertrr marked this pull request as ready for review March 22, 2021 11:41
### What's done:
* Code style
Copy link
Member

@orchestr7 orchestr7 left a comment

Choose a reason for hiding this comment

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

lgtm

@petertrr petertrr merged commit b2538d3 into main Mar 23, 2021
@petertrr petertrr deleted the feature/base-core-cli branch March 23, 2021 08:06
nulls added a commit that referenced this pull request Jun 24, 2022
nulls added a commit that referenced this pull request Jun 27, 2022
nulls added a commit that referenced this pull request Aug 25, 2022
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.

Prepare a native command-line interface for save framework
2 participants