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

docs: README tfsec to trivy migration callout #2020

Merged
merged 5 commits into from
Aug 31, 2023

Conversation

AnaisUrlichs
Copy link
Contributor

Related to the tasks in the following issue #2016

@AnaisUrlichs AnaisUrlichs requested a review from itaysk March 30, 2023 07:44
@github-actions
Copy link

This PR is stale because it has been open 30 days with no activity. Remove stale label or comment or this will be closed in 365 days.

@github-actions github-actions bot added the stale Stale issues will be closed within 7 days of this label being assigned label Apr 29, 2023
@itaysk
Copy link

itaysk commented Apr 29, 2023

@AnaisUrlichs can you please update this and we can merge

@AnaisUrlichs AnaisUrlichs added documentation and removed stale Stale issues will be closed within 7 days of this label being assigned labels Apr 30, 2023
Signed-off-by: AnaisUrlichs <urlichsanais@gmail.com>
Signed-off-by: AnaisUrlichs <urlichsanais@gmail.com>
Signed-off-by: AnaisUrlichs <urlichsanais@gmail.com>
@simar7
Copy link
Member

simar7 commented May 15, 2023

Somewhat related to this PR: Should we have a change notice (IOW a preamble) as part of the tfsec welcome message (when tfsec is run) that also signals the above change? idea by @giorod3

@AnaisUrlichs AnaisUrlichs enabled auto-merge (squash) May 16, 2023 07:49
@AnaisUrlichs AnaisUrlichs disabled auto-merge May 16, 2023 11:07
@simar7
Copy link
Member

simar7 commented Jun 13, 2023

I've cut a PR for the CLI message here #2067

Should we merge this PR? @AnaisUrlichs?

@simar7
Copy link
Member

simar7 commented Aug 17, 2023

ping @AnaisUrlichs?

@AnaisUrlichs
Copy link
Contributor Author

Sorry @simar7 overlooked this -- yes, please -- let's merge it

@simar7 simar7 enabled auto-merge (squash) August 30, 2023 14:21
@simar7 simar7 disabled auto-merge August 31, 2023 15:25
@simar7 simar7 merged commit d7063ef into aquasecurity:master Aug 31, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants