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

Simplification of the analyzer #16

Closed
WaldoFR opened this issue May 10, 2017 · 1 comment
Closed

Simplification of the analyzer #16

WaldoFR opened this issue May 10, 2017 · 1 comment
Assignees
@WaldoFR @dupuisa
Labels
enhancement
Milestone
Version 3.0.0

Comments

@WaldoFR
Copy link

WaldoFR commented May 10, 2017 •
edited
Loading

Inventory

Currently i-Code Analyzer is designed to run an analysis with :

  • A list of IPath to run the analysis on;
  • A configuration based on the plugin preferences;
    It's is also using IPath in it's Violation & FileValue objects returned by the analysis.

Enhancement

We would like to improve it to be able to run an analysis on non-eclipse based thirds, and retrieving only one list of results instead of returning a List<Violation> and a List<FileValue>. This imply to have a common object for both metrics and violations analysis's result and should be realized with a fusion of the two analysis which are independant for now.
@WaldoFR WaldoFR self-assigned this May 10, 2017
@WaldoFR
Copy link
Author

WaldoFR commented May 10, 2017 •
edited
Loading

A dedicated branch AnalyzerSimplification has been created to realize this enhancement.
Build Status

@WaldoFR WaldoFR mentioned this issue May 10, 2017
Closed
@WaldoFR WaldoFR mentioned this issue May 16, 2017
Merged
dupuisa added a commit that referenced this issue May 16, 2017
@dupuisa
Merge pull request #22 from WaldoFR/V3-dev
a012c2f 
Fix #14 & Update #16 using File instead of IPath
This was referenced May 30, 2017
Merged
Merged
dupuisa added a commit that referenced this issue Jun 12, 2017
@dupuisa
Merge pull request #27 from WaldoFR/V3-dev
c2a5884 
Update #16 & Fix #19
@WaldoFR WaldoFR modified the milestone: Version 3.0 Jul 5, 2017
@WaldoFR WaldoFR closed this as completed Jul 5, 2017
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
#16 Using File instead of IPath in Violation and FileValue
8534b59 
Violation and FileValue had both a file attribute which type was IPath.
Changing it into File from java.io.File package imply a lot of little
modifications in files using theses objects.
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
#16 removal of unused IPath import in metric's.
babe540 
Last modification of lex files didn't include the removal of unused
import in metrics.
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
#16 Removal of unused import in test and analyzer
c5d0dcd
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
@dupuisa
Merge pull request #22 from WaldoFR/V3-dev
b0599fc 
Fix #14 & Update #16 using File instead of IPath
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
Update #16 Using an analyzer service to run analysis.
660476e 
Changelog :
- Removal of the old Analyzer in the Analyzer plugin;
- Removal of the analysis handler in the UI plugin
- Add of an Analyzer plugin runnable from any third using
applyRules(List<File>,List<String>,List<String>) and
computeMetrics(List<File>,List<String>,List<String>) indicating files &
languages to run the analysis on. 
- Analyzer is now responsible to run the appropriate files on the good
analyzers
- Add of a new analysis handler in UI sending every selected file to the
analyzer.

The current changes are not complete, javadoc and cleaning task should
be realized to use clean files.
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
Update #16 global test update to use the new analyzer
bf73b6d
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
Update #16 javadoc, code style, and exceptions.
edafe37 
Note : Tests were commented for UI.
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
Update #23 Constructors and attributes #16 function name
8b8deb1 
Checker attributes are now set on new instance. Javadoc added and the
function applyRules was renamed to check.
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
Update #16 Merging metric and rules analyzer
4a8c1b6 
The analyzer service is now only running only one analysis with check
method. This method is now being run two times in the UI plugin, one for
the metric and a second time for the rules. MetricsViews was updated to
take it in consideration and xml export nows add resultValue attribute
when necessary.
Test were modified to also work with theses modifications
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
#Fix 28 & Update #16 adding of fortran 77 files extensions and more.
a78e77c 
Added files extensions to fortran 77 as defined in #28.
Adjusted the Analyzer to handle both situation where the contributor
included the dot before the format extension and not.
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
Update #16 removal of getCheckResult(file)
36798ee 
File in parameter of getCheckResult was useless as in accordance with
the limitation #24, an AbstractChecker is intended only to one file.
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
Update #16 removal of attribute checkResult in AbstractChecker
dda723f
WaldoFR pushed a commit that referenced this issue Jul 13, 2017
@dupuisa
Merge pull request #27 from WaldoFR/V3-dev
0f51c98 
Update #16 & Fix #19
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
cnescatlab#16 Using File instead of IPath in Violation and FileValue
9302a8f 
Violation and FileValue had both a file attribute which type was IPath.
Changing it into File from java.io.File package imply a lot of little
modifications in files using theses objects.
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
cnescatlab#16 removal of unused IPath import in metric's.
433776e 
Last modification of lex files didn't include the removal of unused
import in metrics.
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
cnescatlab#16 Removal of unused import in test and analyzer
da4f1ec
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
@dupuisa
Merge pull request cnescatlab#22 from WaldoFR/V3-dev
aa3f533 
Fix cnescatlab#14 & Update cnescatlab#16 using File instead of IPath
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
Update cnescatlab#16 Using an analyzer service to run analysis.
abddd28 
Changelog :
- Removal of the old Analyzer in the Analyzer plugin;
- Removal of the analysis handler in the UI plugin
- Add of an Analyzer plugin runnable from any third using
applyRules(List<File>,List<String>,List<String>) and
computeMetrics(List<File>,List<String>,List<String>) indicating files &
languages to run the analysis on. 
- Analyzer is now responsible to run the appropriate files on the good
analyzers
- Add of a new analysis handler in UI sending every selected file to the
analyzer.

The current changes are not complete, javadoc and cleaning task should
be realized to use clean files.
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
Update cnescatlab#16 global test update to use the new analyzer
4fc4ac2
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
Update cnescatlab#16 javadoc, code style, and exceptions.
a02c62a 
Note : Tests were commented for UI.
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
Update cnescatlab#23 Constructors and attributes cnescatlab#16 functi…
9257e51 
…on name

Checker attributes are now set on new instance. Javadoc added and the
function applyRules was renamed to check.
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
Update cnescatlab#16 Merging metric and rules analyzer
3530919 
The analyzer service is now only running only one analysis with check
method. This method is now being run two times in the UI plugin, one for
the metric and a second time for the rules. MetricsViews was updated to
take it in consideration and xml export nows add resultValue attribute
when necessary.
Test were modified to also work with theses modifications
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
#Fix 28 & Update cnescatlab#16 adding of fortran 77 files extensions …
771fef0 
…and more.

Added files extensions to fortran 77 as defined in cnescatlab#28.
Adjusted the Analyzer to handle both situation where the contributor
included the dot before the format extension and not.
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
Update cnescatlab#16 removal of getCheckResult(file)
4d8f4bb 
File in parameter of getCheckResult was useless as in accordance with
the limitation cnescatlab#24, an AbstractChecker is intended only to one file.
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
Update cnescatlab#16 removal of attribute checkResult in AbstractChecker
4ccf77c
begarco pushed a commit to begarco/i-CodeCNES that referenced this issue Jan 25, 2020
@dupuisa
Merge pull request cnescatlab#27 from WaldoFR/V3-dev
bee66b8 
Update cnescatlab#16 & Fix cnescatlab#19
begarco pushed a commit that referenced this issue Feb 2, 2020
#16 Using File instead of IPath in Violation and FileValue
1301aa7 
Violation and FileValue had both a file attribute which type was IPath.
Changing it into File from java.io.File package imply a lot of little
modifications in files using theses objects.
begarco pushed a commit that referenced this issue Feb 2, 2020
#16 removal of unused IPath import in metric's.
e680dc9 
Last modification of lex files didn't include the removal of unused
import in metrics.
begarco pushed a commit that referenced this issue Feb 2, 2020
#16 Removal of unused import in test and analyzer
846fa67
begarco pushed a commit that referenced this issue Feb 2, 2020
@dupuisa
Merge pull request #22 from WaldoFR/V3-dev
55ecc2f 
Fix #14 & Update #16 using File instead of IPath
begarco pushed a commit that referenced this issue Feb 2, 2020
Update #16 Using an analyzer service to run analysis.
9de81e9 
Changelog :
- Removal of the old Analyzer in the Analyzer plugin;
- Removal of the analysis handler in the UI plugin
- Add of an Analyzer plugin runnable from any third using
applyRules(List<File>,List<String>,List<String>) and
computeMetrics(List<File>,List<String>,List<String>) indicating files &
languages to run the analysis on. 
- Analyzer is now responsible to run the appropriate files on the good
analyzers
- Add of a new analysis handler in UI sending every selected file to the
analyzer.

The current changes are not complete, javadoc and cleaning task should
be realized to use clean files.
begarco pushed a commit that referenced this issue Feb 2, 2020
Update #16 global test update to use the new analyzer
182c45e
begarco pushed a commit that referenced this issue Feb 2, 2020
Update #16 javadoc, code style, and exceptions.
84fe309 
Note : Tests were commented for UI.
begarco pushed a commit that referenced this issue Feb 2, 2020
Update #23 Constructors and attributes #16 function name
051b7d9 
Checker attributes are now set on new instance. Javadoc added and the
function applyRules was renamed to check.
begarco pushed a commit that referenced this issue Feb 2, 2020
Update #16 Merging metric and rules analyzer
399e460 
The analyzer service is now only running only one analysis with check
method. This method is now being run two times in the UI plugin, one for
the metric and a second time for the rules. MetricsViews was updated to
take it in consideration and xml export nows add resultValue attribute
when necessary.
Test were modified to also work with theses modifications
begarco pushed a commit that referenced this issue Feb 2, 2020
#Fix 28 & Update #16 adding of fortran 77 files extensions and more.
2fb862b 
Added files extensions to fortran 77 as defined in #28.
Adjusted the Analyzer to handle both situation where the contributor
included the dot before the format extension and not.
begarco pushed a commit that referenced this issue Feb 2, 2020
Update #16 removal of getCheckResult(file)
a78b97f 
File in parameter of getCheckResult was useless as in accordance with
the limitation #24, an AbstractChecker is intended only to one file.
begarco pushed a commit that referenced this issue Feb 2, 2020
Update #16 removal of attribute checkResult in AbstractChecker
e57880d
begarco pushed a commit that referenced this issue Feb 2, 2020
@dupuisa
Merge pull request #27 from WaldoFR/V3-dev
eecdb33 
Update #16 & Fix #19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@WaldoFR WaldoFR

@dupuisa dupuisa

Labels
enhancement
Projects
None yet
Milestone
Version 3.0.0
Development

No branches or pull requests
2 participants
@WaldoFR @dupuisa