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

[INTERNAL] Extension handling groundwork #46

Merged
merged 6 commits into from
Oct 16, 2018

Conversation

RandomByte
Copy link
Member

@RandomByte RandomByte commented Oct 12, 2018

Identify and apply extensions in dependency tree so that they can influence the processing of a project.

Implements foundation for "extensions" as required in RFC 0001 (Type Extensibility) and RFC 0004 (Simple Build Extensibility)

Implements shim extensions as defined in RFC 0002 (Project Shims)

See [RFC0001](SAP/ui5-tooling#4).

Identify and apply extensions in dependency tree so that they can
influence the actual project processing.
Also enhance extension metadata checks
Used in npm scripts. Already a dependency in all other tooling modules
And fix shimed dependency and collection handling
@coveralls
Copy link

Pull Request Test Coverage Report for Build 198

  • 142 of 179 (79.33%) changed or added relevant lines in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+2.7%) to 71.609%

Changes Missing Coverage Covered Lines Changed/Added Lines %
lib/projectPreprocessor.js 142 179 79.33%
Files with Coverage Reduction New Missed Lines %
lib/projectPreprocessor.js 2 75.82%
Totals Coverage Status
Change from base Build 187: 2.7%
Covered Lines: 275
Relevant Lines: 363

💛 - Coveralls

@RandomByte
Copy link
Member Author

@matz3 ping

@RandomByte RandomByte changed the title Extension handling groundwork [INTERNAL] Extension handling groundwork Oct 16, 2018
@RandomByte RandomByte merged commit e0ecb7f into master Oct 16, 2018
@RandomByte RandomByte deleted the extension-handling-groundwork branch October 16, 2018 15:33
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.

None yet

3 participants