Skip to content

Merge 1.X.X to 2.0 as new version 2.1.0 #98

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

Closed
wants to merge 57 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
57 commits
Select commit Hold shift + click to select a range
a15f700
Fixes for issues 61, 62, and 64. New patch marked version v1.0.2.
rahlk Nov 5, 2024
29fd643
Update main.yml
rahlk Nov 5, 2024
3e73301
Update main.yml
rahlk Nov 5, 2024
61fc5b8
Update main.yml
rahlk Nov 7, 2024
27c09f9
Add root pom path to the CLI
rahlk Nov 7, 2024
ffdef39
Add root pom path to the CLI
rahlk Nov 7, 2024
7a4fb10
Merge pull request #68 from IBM/remove-no-copy-deps-and-add-path-to-r…
rahlk Nov 7, 2024
bb56aca
Merge pull request #67 from IBM/mark-a-release-on-1.X.X
rahlk Nov 7, 2024
934a54f
Add root pom path to the CLI
rahlk Nov 7, 2024
1acec4a
Add root pom path to the CLI
rahlk Nov 7, 2024
7d740b9
Add root pom path to the CLI
rahlk Nov 7, 2024
0925555
mvnw now looks for absolute path
rahlk Nov 11, 2024
bb3b2b6
Merge pull request #71 from IBM/1.0.4
rahlk Nov 11, 2024
b9d53a9
Upgrade to WALA 1.6.7
rahlk Nov 11, 2024
17a3bff
Merge pull request #72 from IBM/1.0.4
rahlk Nov 11, 2024
21ef81a
mvn.bat -> mvn.cmd
rahlk Nov 13, 2024
75944f1
Merge pull request #75 from IBM/1.0.5
rahlk Nov 13, 2024
ff079c5
Fix issue 76--symbol table is none.
rahlk Nov 19, 2024
1c07358
Merge pull request #77 from IBM/1.0.6
rahlk Nov 19, 2024
7b7a451
Bump version to 1.0.6
rahlk Nov 19, 2024
9946546
Fix issue #79
rahlk Dec 3, 2024
b32343b
Merge pull request #80 from IBM/1.0.7-dev
rahlk Dec 3, 2024
e311d96
Fix issue #79
rahlk Dec 3, 2024
42caced
Merge branch '1.X.X' into 1.0.7
rahlk Dec 3, 2024
cb285fa
Merge pull request #81 from IBM/1.0.7
rahlk Dec 3, 2024
1ad2b92
Fix issue #83
rahlk Dec 12, 2024
392d12a
Cyclomatic complexity computation for analysis level 1
sinha108 Jan 13, 2025
ebe64b6
Added catch clause count to computation of cyclomatic complexity for …
sinha108 Jan 13, 2025
83d55e0
Clean up callable declaration string to remove embedded comments.
sinha108 Jan 15, 2025
5d14706
Fixing issue 87 with maven wrapper
rahlk Jan 15, 2025
f30e384
Fixing issue 87 with maven wrapper
rahlk Jan 15, 2025
8253562
Merge pull request #89 from IBM/cyclomatic-complexity-analysis-level1
rahlk Jan 15, 2025
6e745c4
Fixing issue 87 with maven wrapper
rahlk Jan 15, 2025
93e5efb
Fixing issue 87 with maven wrapper
rahlk Jan 15, 2025
0dcde52
Fixing issue 87 with maven wrapper
rahlk Jan 15, 2025
ff1184b
Throw a more discriptive error message when build commands fail.
rahlk Jan 17, 2025
8bdbace
Merge pull request #92 from IBM/fix-issue-87
rahlk Jan 18, 2025
e94761f
Don't fail on build errors during download dependencies phase
rahlk Jan 20, 2025
65ebcf2
Merge pull request #94 from IBM/feature-request-on-issue-93
rahlk Jan 20, 2025
10b48d8
Update command checker to proactively set maven/gradle. Update make -…
rahlk Jan 21, 2025
3433add
Add more test cases
rahlk Jan 21, 2025
5a4ea53
Add more test cases
rahlk Jan 21, 2025
a790730
Update command checker to proactively set maven/gradle. Update make -…
rahlk Jan 21, 2025
7b38a27
Update command checker to proactively set maven/gradle. Update make -…
rahlk Jan 21, 2025
98ea868
Add Integration Tests to replicate behaviour.
rahlk Jan 22, 2025
2310996
Add Integration Tests to replicate behaviour.
rahlk Jan 22, 2025
c3bbc67
Refactor Integration Tests.
rahlk Jan 23, 2025
69c2067
Refactor Integration Tests.
rahlk Jan 23, 2025
5355c2f
Fixed failing Integration tests
rahlk Jan 24, 2025
575f6a7
Enhance the lookup for jmods folder. We now use a recursive folder se…
rahlk Jan 24, 2025
2abc7d8
Merge pull request #96 from IBM/fix-issue-95
rofrano Jan 24, 2025
5cf8398
Set Java version to 11
rahlk Jan 24, 2025
6180850
Attempt fix for issue 74 by moving dowloaded dependencies to the proj…
rahlk Jan 31, 2025
231e40a
Attempt fix for issue 74 by moving dowloaded dependencies to the proj…
rahlk Jan 31, 2025
86c5bca
Update quickstart example with a bug fix.
rahlk Feb 3, 2025
412d3d4
Merge pull request #99 from IBM/fix-issue-74
rahlk Feb 3, 2025
1af764f
Remove dangling file
rahlk Feb 3, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Clean up callable declaration string to remove embedded comments.
Signed-off-by: Saurabh Sinha <sinha108@gmail.com>
  • Loading branch information
sinha108 committed Jan 15, 2025
commit 83d55e090908ec92af2961da84671e0e5a82c99c
4 changes: 3 additions & 1 deletion src/main/java/com/ibm/cldk/SymbolTable.java
Original file line number Diff line number Diff line change
Expand Up @@ -273,7 +273,9 @@ private static Pair<String, Callable> processCallableDeclaration(CallableDeclara

// add the complete declaration string, including modifiers, throws, and
// parameter names
callableNode.setDeclaration(callableDecl.getDeclarationAsString(true, true, true).strip());
callableNode.setDeclaration(callableDecl
.getDeclarationAsString(true, true, true)
.strip().replaceAll("//.*\n", ""));

// add information about callable parameters: for each parameter, type, name,
// annotations,
Expand Down