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

D assistant take 2 #355

Open
wants to merge 24 commits into
base: main
Choose a base branch
from
Open

D assistant take 2 #355

wants to merge 24 commits into from

Conversation

pyth0n1c
Copy link
Contributor

@pyth0n1c pyth0n1c commented Jan 23, 2025

assistant work
This contains a Proof of Concept and initial progress for discussion purposes.
DO NOT MERGE AT THIS TIME

@pyth0n1c pyth0n1c added the Draft label Jan 27, 2025
resolving circular
import issues
in the directory. No need to pass around the type
of an object, we have the Object class!
to all SecurityContentObjects to aid
in parsing.
to support deprecation of
more than just detection
content. Also, enforce
the presence of deprecation
info in the deprecation_yml
when content is marked as
deprecated.
dist that looks appropriate. However lots of things
are still a bit sloppy. We will need some comment
on the contents of everything.
we should have removed but did not
@patel-bhavin
Copy link
Contributor

Hi Eric, a few comments :

  • Since we are creating this automatically at build time , should we drop this lookup file in dist/ and throw in a stanza in transforms? We can also maintain this in the lookups directory as the website will need this file as well.

  • Lets add the name of the replacement detection name to the CSV (it would better for the dashboard)

migration_guide from CSV file output
anymore. The folder does not
exist in the repo, and we get
an error if that folder does not
exist.
around deprecated vs removed content.
Improvements to the generation of
the deprecated_csv lookup file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants