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.

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

Add parser for CCES (CrossCore Embedded Studio) #1566

Merged
merged 3 commits into from
Aug 2, 2023

Conversation

arthur-devialet
Copy link
Contributor

@arthur-devialet arthur-devialet commented Jul 31, 2023

This integrates jenkinsci/analysis-model#937

Testing done

  • Most testing done on analysis-model side
  • Updated here ParsersITest.java, but I could not find how to run it yet

Submitter checklist

  • Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
  • Ensure that the pull request title represents the desired changelog entry
  • Please describe what you did
  • Link to relevant issues in GitHub or Jira
  • Link to relevant pull requests, esp. upstream and downstream changes
  • Ensure you have provided tests - that demonstrates feature works or fixes the issue

@arthur-devialet arthur-devialet changed the title Add parser for CCES (CrossCore Embedded Studio from Analog Devices) Add parser for CCES (CrossCore Embedded Studio) Jul 31, 2023
@uhafner uhafner added the enhancement Enhancement of existing functionality label Aug 1, 2023
Copy link
Member

@uhafner uhafner left a comment

Choose a reason for hiding this comment

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

You need to merge with #1568 once that build is green.

@codecov
Copy link

codecov bot commented Aug 2, 2023

Codecov Report

Merging #1566 (489ef7b) into master (b6a12a6) will increase coverage by 0.10%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master    #1566      +/-   ##
============================================
+ Coverage     81.74%   81.85%   +0.10%     
- Complexity     1401     1404       +3     
============================================
  Files           248      249       +1     
  Lines          5335     5339       +4     
  Branches        396      396              
============================================
+ Hits           4361     4370       +9     
+ Misses          854      850       -4     
+ Partials        120      119       -1     
Files Changed Coverage Δ
...ins/analysis/warnings/CrossCoreEmbeddedStudio.java 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@uhafner uhafner merged commit a608d61 into jenkinsci:master Aug 2, 2023
32 checks passed
@arthur-devialet arthur-devialet deleted the arthur/add-CCES-parser branch August 2, 2023 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Enhancement of existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants