Skip to content

Load order create configuration#443

Merged
ktalebian merged 4 commits intov4-betafrom
load-order-create-configuration
Apr 29, 2021
Merged

Load order create configuration#443
ktalebian merged 4 commits intov4-betafrom
load-order-create-configuration

Conversation

@ktalebian
Copy link
Copy Markdown
Contributor

Enforce load-order for enabling plugins:

  • Adding new plugins are appended to the end of the list
  • Updating an existing plugin's version updates the plugin in place (preserves the order)


const getLatestPluginVersion = async (name: string): Promise<PluginVersionResource | null> => {
return versionsClient.latest(name);
try {
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This was for e2e test the first time it runs.

@ktalebian ktalebian requested review from a team and ahcai April 28, 2021 04:39
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 28, 2021

Codecov Report

Merging #443 (d6f9183) into v4-beta (c71e87e) will increase coverage by 0.06%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##           v4-beta     #443      +/-   ##
===========================================
+ Coverage    84.86%   84.91%   +0.06%     
===========================================
  Files          144      144              
  Lines         3513     3526      +13     
  Branches       583      583              
===========================================
+ Hits          2981     2994      +13     
  Misses         433      433              
  Partials        99       99              
Impacted Files Coverage Δ
...ins-api-toolkit/src/scripts/createConfiguration.ts 91.30% <100.00%> (+2.02%) ⬆️

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 c71e87e...d6f9183. Read the comment docs.

);
const list: string[] = option.addPlugins;
const list: string[] = [];
// = option.addPlugins;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

revert?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Na gotta delete this actually. No longer used. The implementation has changed.

@ktalebian ktalebian merged commit 96dfbf5 into v4-beta Apr 29, 2021
@ktalebian ktalebian deleted the load-order-create-configuration branch April 29, 2021 16:59
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.

2 participants