-
Notifications
You must be signed in to change notification settings - Fork 26
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
plugin complement #409
Merged
Merged
plugin complement #409
Changes from 40 commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
56b1cce
Created class for managing plugins
christophe-david f731f6e
Plugins loading is now delayed to the moment where it is needed.
christophe-david 6a652e0
Fixed module import for plugins
christophe-david 4099f53
Moved run_system() from tests/ folder into private FAST-OAD code.
christophe-david c628235
Updated pyproject.toml
christophe-david 67fd6c4
Fused PluginManager into FastoadLoader
christophe-david 6deac71
Added configuration loading in FastoadLoader
christophe-david f816233
Minor fix in BundleLoader
christophe-david bcf6418
Better check of package contents
christophe-david f0184ce
New plugin definition
christophe-david c42b1d1
Added method to get available sample configuration files.
christophe-david ac7be25
generate_configuration_file() now handles plugins
christophe-david 2a7a232
Now generate_configuration_file() does not require plugin name if onl…
christophe-david 1e00605
Factorized dummy plugins for tests
christophe-david 81f28a6
Removed FastoadLoader.get_configuration_file_list()
christophe-david f9b76c5
Added dummy plugin with only notebooks (bug fix in get_plugin_configu…
christophe-david 062b289
Added command to provide plugin information
christophe-david 59b1507
Minor update
christophe-david 1960322
reorganized fixtures for dummy plugins
christophe-david 1f82469
Changed identification of plugins
christophe-david 5fb73b0
Now handling "no plugin" case in configuration generation.
christophe-david 4bb7592
Updated CLI subcommand "gen_conf"
christophe-david d974379
Some housekeeping
christophe-david d7351f4
Changed identifier for fastoad plugins
christophe-david a66ac5c
Added class DistributionPluginDefinition
christophe-david 0587c8a
Some cleaning
christophe-david 6e647f6
Replaced usage of pkg_resources by importlib.metadata (as it is now a…
christophe-david 78fccb9
Introduced ConfigurationFileInfo class.
christophe-david 5d594da
Refactoring
christophe-david ad7e6ba
Refactoring again
christophe-david 44391ae
Added method to get notebook folder information
christophe-david 8f6863b
Updated command "fastoad notebooks"
christophe-david ec478c4
Updated Codacy settings
christophe-david a7961d6
Added api.generate_notebooks()
christophe-david ec1c0e6
Added api.get_plugin_information()
christophe-david 7cf71af
Linting
christophe-david c1d202f
Updated documentation of CLI usage
christophe-david 57aaa36
Updated documentation
christophe-david dbd4727
Updated documentation
christophe-david 7bb48ed
Moved get_xfoil. Enhanced PackageReader (better check for existence).
christophe-david a64d130
Updated sphinx doc.
christophe-david 61cab9d
Updated sphinx doc.
christophe-david File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In the section "Publishing" I would underline that to publish on PyPi the package name will be the one provided in the field
name
in thepyproject.toml
. Therefore, the publisher shall check that the chosen package name is not already taken on PyPi.There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agreed. Done in a64d130.