-
Notifications
You must be signed in to change notification settings - Fork 21
feat: remove splunk-appinspect dependency #464
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
Merged
Merged
Changes from all commits
Commits
Show all changes
19 commits
Select commit
Hold shift + click to select a range
d6e8a89
ci: apply concurrency for build-test-release job
f3701ae
refactor: use conf-parser in EventtypeParser instead of splunk-appins…
artemrys 6cbc550
refactor: use conf-parser in SavedsearchesParser instead of splunk-ap…
artemrys da8db9c
refactor: use conf-parser in TransformsParser instead of splunk-appin…
artemrys b585df0
refactor: cache parsed results
artemrys 345ac95
refactor: use conf-parser in PropsParser instead of splunk-appinspect
artemrys 0c5c25b
refactor: remove splunk-appinspect from AddonParser
artemrys 4b3e4dc
docs: remove splunk-appinspect mentions from PropsParser
artemrys ba71246
test: empty stanza in savedsearches.conf
artemrys 400e62c
refactor: use new EventtypeParser in cim_field_report
artemrys 6abd3b5
refactor: use conf-parser in eventgen.conf parser
artemrys fa19546
chore: remove create_new_eventgen
artemrys c2df52c
feat: remove splunk_appinspect dependency
artemrys f4403bf
chore: rename to pytest_splunk_addon_data
artemrys 8e7b8e3
chore: style the docs pagee
artemrys 9a640c5
ci: restructure tests
artemrys 2f149b7
ci: not run tests with Splunk 8.0
artemrys 260b0f6
refactor: move some public methods to be private
artemrys 0f7a326
ci: add debug-statements to pre-commit configuration
artemrys File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -42,13 +42,11 @@ Developing | |
|
|
||
| Note: Must install docker desktop, vscode or pycharm pro optional | ||
|
|
||
| Note2: Appinspect requires libmagic verify this has been installed correctly each time a new workstation/vm is used https://dev.splunk.com/enterprise/docs/releaseapps/appinspect/splunkappinspectclitool/installappinspect | ||
|
|
||
| .. code:: bash | ||
|
|
||
| $ git clone --recurse-submodules -j8 [email protected]:splunk/pytest-splunk-addon.git | ||
|
|
||
| $ #setup python venv must be 3.7 | ||
| $ #setup python venv must be 3.7 | ||
| $ /Library/Frameworks/Python.framework/Versions/3.7/bin/python3 -m venv .venv | ||
|
|
||
| $ source .venv/bin/activate | ||
|
|
@@ -57,8 +55,6 @@ Note2: Appinspect requires libmagic verify this has been installed correctly eac | |
|
|
||
| $ pip3 install -r requirements.txt | ||
|
|
||
| $ pip3 install https://download.splunk.com/misc/appinspect/splunk-appinspect-latest.tar.gz | ||
|
|
||
| $ python setup.py develop | ||
|
|
||
|
|
||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.