Skip to content

Conversation

@crankynetman
Copy link
Collaborator

No description provided.

samoehlert and others added 30 commits November 8, 2024 11:25
… actions

start pointing at develop as our default branch to match reality
SAST is already running via the settings on this repo
also turn off our testing branch push runs
works in conjunction with the if on the last step to run always, excpet only publish the docs on the main branch
…on in our local settings instead of inheriting them
…emp-fix

ci(docs): only build docs on main
grigorescu and others added 25 commits November 23, 2024 14:46
Fixes pytest warning: `No directory at:
/home/runner/work/SCRAM/SCRAM/staticfiles/`

---------

Co-authored-by: Chris Cummings <chriscummings@es.net>
A couple extra things:

* Add a Behave test with Python 3.13, similar to the pytest one
* Make the Python version an argument to the Dockerfiles
This lets you run prod with a local postgres instance. We prolly need to
document the vault stuff, but I think that's better suited for #88 to
handle.

I *think* this MR is ready to rock, it's deployed and working on
`scram-pentest.ocean.cu-es.net` presently.
@grigorescu grigorescu enabled auto-merge (squash) December 23, 2024 19:03
@github-actions
Copy link

github-actions bot commented Dec 23, 2024

File Coverage
All files 81%
config/consumers.py 71%
config/urls.py 69%
config/settings/base.py 70%
config/settings/local.py 72%
scram/route_manager/models.py 70%
scram/route_manager/views.py 78%
scram/route_manager/api/serializers.py 73%
scram/route_manager/api/views.py 83%
scram/templates/403.html 91%
scram/templates/404.html 91%
scram/templates/base.html 99%

Minimum allowed coverage is 50%

Generated by 🐒 cobertura-action against 6810e74

Copy link
Collaborator

@samoehlert samoehlert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good to get this released

@grigorescu grigorescu merged commit 740dd78 into main Dec 23, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants