Skip to content

initial feature-manifest language setup#4588

Merged
tarikeshaq merged 3 commits intomainfrom
nimbus-fml-init
Oct 19, 2021
Merged

initial feature-manifest language setup#4588
tarikeshaq merged 3 commits intomainfrom
nimbus-fml-init

Conversation

@tarikeshaq
Copy link
Copy Markdown
Contributor

fixes https://mozilla-hub.atlassian.net/browse/SDK-400

Super minimal new crate setup, so others have something to start working off of, happy to move this elsewhere / take any feedback, but lets get this merged so we all have can work off the same crate, I'm not married to any type of project setup here, so feel free to change things up in future PRs!

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 19, 2021

Codecov Report

Merging #4588 (f874a8a) into main (2b3ee03) will decrease coverage by 0.06%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #4588      +/-   ##
==========================================
- Coverage   80.69%   80.63%   -0.07%     
==========================================
  Files          48       51       +3     
  Lines        5217     5221       +4     
==========================================
  Hits         4210     4210              
- Misses       1007     1011       +4     
Impacted Files Coverage Δ
components/support/nimbus-fml/src/lib.rs 0.00% <0.00%> (ø)
components/support/nimbus-fml/src/main.rs 0.00% <0.00%> (ø)
components/support/nimbus-fml/src/parser.rs 0.00% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2b3ee03...f874a8a. Read the comment docs.

Copy link
Copy Markdown
Member

@travis79 travis79 left a comment

Choose a reason for hiding this comment

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

And thus Nimbus-FML was born :)

.author("Nimbus SDK Engineering")
.about("A tool to generate code using an experiment feature manifest")
.arg(
Arg::with_name("manifest")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Depending on how we go about it, we may have a need to enter multiple manifest files here.

@tarikeshaq tarikeshaq merged commit d0d4552 into main Oct 19, 2021
@tarikeshaq tarikeshaq deleted the nimbus-fml-init branch October 19, 2021 16:17
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