Skip to content
This repository has been archived by the owner on Apr 18, 2023. It is now read-only.

Sanity check output #44

Open
vpetersson opened this issue Nov 7, 2021 · 2 comments
Open

Sanity check output #44

vpetersson opened this issue Nov 7, 2021 · 2 comments
Assignees
@robputt

Comments

@vpetersson
Copy link
Contributor

We need to sanity check output before we copy it into the bucket.
@shawaj
Copy link
Member

shawaj commented Nov 7, 2021

Was going to create an issue for this too 👍👌

@robputt robputt self-assigned this Nov 11, 2021
@vpetersson
Copy link
Contributor Author

@robputt can you tackle this one? I'm thinking a simple regex check on these variables individually should do the trick.

vpetersson pushed a commit that referenced this issue Feb 8, 2022
Refs #44. Adds initial sanity check and strict bash mode
139a39f
@vpetersson vpetersson mentioned this issue Feb 8, 2022
Merged
vpetersson added a commit that referenced this issue Feb 8, 2022
@vpetersson @marvinmarnold
Refs #44. Adds initial sanity check and strict bash mode (#64)
ae59a3e 
* Refs #44. Adds initial sanity check and strict bash mode

* fix: only upload snapshots once

Co-authored-by: Marvin Arnold <marvin@geeky.rocks>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@robputt robputt

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@vpetersson @shawaj @robputt