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.

Sign up for GitHub

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

Jump to bottom

Refactor plugins #1032

Merged
merged 27 commits into from
Oct 1, 2024
Merged

Refactor plugins #1032

merged 27 commits into from
Oct 1, 2024

Conversation

narekhovhannisyan
Copy link
Member

Types of changes

  • Enhancement (project structure, spelling, grammar, formatting)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

A description of the changes proposed in the Pull Request

  • Integrate plugin factory into IF framework.

manushak and others added 26 commits September 19, 2024 20:48
@manushak
feat(builtins): refactor , and plugins
9dddd5d
@manushak
feat(builtins): refactor plugins
7e5d867
@manushak
test(builtins): temporary skip refcatored plugins' tests
4ff5d3f
@manushak
feat(builtins): refactor sci-embodied plugin
a9e3fec
@manushak
test(builtins): temporary skip sci-embodeied tests
f57fd97
@manushak
fix(util): fix ts complainig related to unnecessary assigning pluginP…
17f823b 
…arameters to {}
@manushak
feat(builtins): refactor sum plugin
ba6b1c8
@manushak
test(builtins): temporary skip sum plugin tests
c0f5572
@narekhovhannisyan
chore(src): merge branch 'main' refactor-plugins
d11400f
@manushak
fix(builtins): fix plugins config validation issue
8bff085
@manushak
docs(builtins): update readme files of plugins
2f8eaec
@manushak
docs(builtins): update time-converter readme file
4fee1ac
@manushak
docs(src): update migration guide
3cee69c
@manushak
test(builtins): update plugins' tests
577f76f
@narekhovhannisyan
chore(src): fetch latest changes from refactor-plugins
cb7cde7
@narekhovhannisyan
test(builtins): remove skip from mock observation
04aa551
@narekhovhannisyan
test(builtins): tune unit tests for time sync
7bce20d
@narekhovhannisyan
feat(builtins): rewrite regex with plugin factory
7963d25
@narekhovhannisyan
feat(builtins): rewrite shell with plugin factory
c1e019e
@narekhovhannisyan
feat(builtins): rewrite time-sync with plugin factory
5f903fc
@manushak
test(builtins): fix tests for time-sync and sci-embodied plugins
23e0742
@manushak
fix(builtins): fix arithmetic feature logic in the coeffient and sci-…
93f6b5f 
…embodied pluings
@narekhovhannisyan
revert(types): remove time sync
01646f1
@narekhovhannisyan
fix(builtins): minor issues in time sync, import types from if-core
4f7361e
@narekhovhannisyan
feat(builtins): drop empty metadata objects
3cc0472
@narekhovhannisyan
feat(package): update if-core dependency version
bea643b
@narekhovhannisyan narekhovhannisyan self-assigned this Sep 30, 2024
@narekhovhannisyan narekhovhannisyan linked an issue Oct 1, 2024 that may be closed by this pull request
Plugin Factory #989
Closed
2 tasks
@zanete zanete linked an issue Oct 1, 2024 that may be closed by this pull request
Update built-in, ecoCi and GH plugins to work with the new if-core interface #1017
Closed
8 tasks
This was unlinked from issues Oct 1, 2024
Plugin Factory #989
Closed
Update built-in, ecoCi and GH plugins to work with the new if-core interface #1017
Closed
@narekhovhannisyan narekhovhannisyan merged commit cf83951 into main Oct 1, 2024
2 checks passed
@narekhovhannisyan narekhovhannisyan deleted the refactor-plugins branch October 1, 2024 15:23
@manushak manushak linked an issue Oct 3, 2024 that may be closed by this pull request
Update built-in, ecoCi and GH plugins to work with the new if-core interface #1017
Closed
8 tasks
This was referenced Oct 4, 2024
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@manushak manushak manushak approved these changes

@jmcook1186 jmcook1186 jmcook1186 approved these changes

Assignees

@narekhovhannisyan narekhovhannisyan

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Update built-in, ecoCi and GH plugins to work with the new if-core interface
3 participants
@narekhovhannisyan @manushak @jmcook1186