Skip to content

Conversation

@ghost
Copy link

@ghost ghost commented Mar 8, 2022

This PR has 2 significant changes:

Minor changes (see the list of commits):

Closes #475

Artem Rys and others added 9 commits March 8, 2022 12:05
BREAKING CHANGE: eventgen.conf is not supported anymore, please migrate to pytest-splunk-addon-data.conf

* feat: remove splunk-appinspect dependency (#464)
* refactor: use conf-parser in EventtypeParser instead of splunk-appinspect
* refactor: use conf-parser in SavedsearchesParser instead of splunk-appinspect
* refactor: use conf-parser in TransformsParser instead of splunk-appinspect
* refactor: use conf-parser in PropsParser instead of splunk-appinspect
* refactor: remove splunk-appinspect from AddonParser
* docs: remove splunk-appinspect mentions from PropsParser
* test: empty stanza in savedsearches.conf
* refactor: use new EventtypeParser in cim_field_report
* refactor: use conf-parser in eventgen.conf parser
* chore: remove create_new_eventgen
* feat: remove splunk_appinspect dependency
* chore: rename to pytest_splunk_addon_data
* chore: style the docs pagee
* ci: restructure tests
* ci: not run tests with Splunk 8.0
* refactor: move some public methods to be private
* ci: add debug-statements to pre-commit configuration
* docs: remove release_history pagei
* ci: upload coverage to codecov
* chore: remove all eventgen signs
* ci: try with parallel runs

* ci: do not install additional deps

* ci: reorganize long-running tests to make tests faster

* ci: do not need to upload dist when publishing

* ci: remove snyk

* ci: upload NOTICE file
@codecov-commenter
Copy link

codecov-commenter commented Mar 8, 2022

Codecov Report

❗ No coverage uploaded for pull request base (main@1035f22). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #574   +/-   ##
=======================================
  Coverage        ?   94.88%           
=======================================
  Files           ?       50           
  Lines           ?     2619           
  Branches        ?        0           
=======================================
  Hits            ?     2485           
  Misses          ?      134           
  Partials        ?        0           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1035f22...b2d359e. Read the comment docs.

@github-actions
Copy link
Contributor

github-actions bot commented Mar 8, 2022

Expected release notes (by @arys-splunk)

features:
allow *.xml (beside *.log) files as sample input files (#550) (81e654b)
remove splunk-appinspect dependency (#499) (c7caa68)

fixes:
revert helmut's library clean up (1035f22)

others (will not be included in Semantic-Release notes):
semgrep and misspell fixes (b2d359e)
update development instructions (#564) (3a43f48)
reorganize how doc are being handled in CI (#563) (a33c8b5)
remove pytest-mock as not used (57f7a9a)
fix semgrep issues (#558) (176aa76)
remove Ederson from owners list [ci skip] (#559) (557c2ec)
make tests run in parallel (#553) (caddde4)
move test_cim_report.py to unit tests (#554) (50de532)
remove test_events_with_untokenised_values from RequirementBasic (#543) (23deab5)
remove matrix option from test-splunk-matrix step (#545) (5464510)

  • I hereby acknowledge these release notes are 🥙 AWESOME 🥙

@hsekowski-splunk hsekowski-splunk self-requested a review March 8, 2022 13:32
@ghost ghost merged commit c860c14 into main Mar 8, 2022
@ghost ghost deleted the release-3.0.0 branch March 8, 2022 13:41
@github-actions github-actions bot locked and limited conversation to collaborators Mar 8, 2022
@srv-rr-github-token
Copy link
Contributor

🎉 This PR is included in version 3.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove support for eventgen.conf

4 participants