Skip to content

[ti_anomali] Remove dotted-yaml keys#7803

Merged
andrewkroh merged 1 commit intoelastic:mainfrom
andrewkroh:ti_anomali/dotted-yaml-cleanup
Sep 13, 2023
Merged

[ti_anomali] Remove dotted-yaml keys#7803
andrewkroh merged 1 commit intoelastic:mainfrom
andrewkroh:ti_anomali/dotted-yaml-cleanup

Conversation

@andrewkroh
Copy link
Copy Markdown
Member

In preparation for upgrading to package-spec 3.0.0, this replaces YAML keys containing dots with an unambiguous YAML form. It also removes the empty _meta object from the transform definition.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

@elasticmachine
Copy link
Copy Markdown

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

In preparation for upgrading to package-spec 3.0.0, this replaces YAML keys containing dots with an unambiguous YAML form. It also removes the empty _meta object from the transform definition.
@elasticmachine
Copy link
Copy Markdown

elasticmachine commented Sep 13, 2023

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2023-09-13T16:55:11.378+0000

  • Duration: 17 min 21 sec

Test stats 🧪

Test Results
Failed 0
Passed 11
Skipped 0
Total 11

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

@elasticmachine
Copy link
Copy Markdown

🌐 Coverage report

Name Metrics % (covered/total) Diff
Packages 100.0% (1/1) 💚
Files 100.0% (1/1) 💚
Classes 100.0% (1/1) 💚
Methods 100.0% (13/13) 💚
Lines 91.071% (357/392) 👎 -7.774
Conditionals 100.0% (0/0) 💚

@andrewkroh andrewkroh merged commit 6da6c61 into elastic:main Sep 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Integration:ti_anomali Anomali ThreatStream

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants