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.

Sign up for GitHub

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

Jump to bottom

Deploy to test #1790

Merged
merged 102 commits into from
Nov 20, 2023
Merged

Deploy to test #1790

merged 102 commits into from
Nov 20, 2023

Conversation

thekaveman
Copy link
Member

  • Azure fileshare changes (off by default unless configured in settings)
  • Admin changes (off by default unless configured in settings)
  • SBMTD Mobility Pass config

thekaveman and others added 30 commits September 19, 2023 14:30
@thekaveman
chore(deploy): pip freeze to a static file
0a961c2 
similar to having the SHA, will help confirm changes and debug
@machikoyasuda
chore(docker): remove platform designation, no longer needed
fe2f5da
@thekaveman
chore(tests): use coverage directly to run tests
59f2a1f 
no need for the pytest-cov wrapper library
@thekaveman
chore(ci): add coverage comment step
fba49f4
@thekaveman
chore(tests): rename sample classes
1a3654c 
avoids warning that pytest can't find test cases to collect
@thekaveman
fix(coverage): run with relative files on
c3b8720 
see https://coverage.readthedocs.io/en/6.2/config.html#config-run-relative-files

needed by coverage comment: https://github.com/py-cov-action/python-coverage-comment-action?tab=readme-ov-file#setup
@thekaveman
chore(coverage): only analyze app code
7c292f9
@indexing
docs: adding veterans use case
3fa9c9e
@pre-commit-ci
chore(pre-commit): autofix run
95362a0
@indexing @angela-tran
Update docs/use-cases/Veterans.md
9251453 
Co-authored-by: Angela Tran <angela@compiler.la>
@pre-commit-ci
chore(pre-commit): autofix run
80564d2
@indexing
docs: replacing diagram image with mermaid code
597eb50
@indexing
docs: removing clickup garbage in some link URLs
dcb955a
@indexing
docs: removed bolding on h2 headings
60ad329 
This styling came with the .md file export from ClickUp. We definitely need to be aware that the Markdown in these exports isn't clean and should be reviewed carefully.
@machikoyasuda
Docker: Revert platform change (#1743)
85a130d
@pre-commit-ci
chore(pre-commit): autoupdate hooks
9d6f555 
updates:
- [github.com/Riverside-Healthcare/djLint: v1.32.1 → v1.34.0](djlint/djLint@v1.32.1...v1.34.0)
@indexing @machikoyasuda
docs: update section title to use sentence case
d3f78ac 
Co-authored-by: machiko <machiko@compiler.la>
@indexing @machikoyasuda
docs: update section title to use sentence case
3d5aa81 
Co-authored-by: machiko <machiko@compiler.la>
@indexing @thekaveman
docs: update section title to use sentence case
0e07007 
Co-authored-by: Kegan Maher <kegan@compiler.la>
@machikoyasuda
Docs: adding veterans use case (#1748)
a792131
@machikoyasuda
chore(pre-commit): autoupdate hooks (#1749)
f6a2974
@thekaveman
chore(deploy): use pipdeptree instead of pip freeze
280e9d9
@thekaveman
Tests: add coverage comments to PRs (#1745)
e22bef2
@dependabot
chore(deps-dev): bump cypress from 13.2.0 to 13.3.0 in /tests/cypress
cde410a 
Bumps [cypress](https://github.com/cypress-io/cypress) from 13.2.0 to 13.3.0.
- [Release notes](https://github.com/cypress-io/cypress/releases)
- [Changelog](https://github.com/cypress-io/cypress/blob/develop/CHANGELOG.md)
- [Commits](cypress-io/cypress@v13.2.0...v13.3.0)

---
updated-dependencies:
- dependency-name: cypress
  dependency-type: direct:development
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@thekaveman
Chore: add pip metadata file to app container (#1733)
4ab5416
@machikoyasuda
fix: do not show scrollbar when not necessary
db7d8c8
@machikoyasuda
chore(deps-dev): bump cypress from 13.2.0 to 13.3.0 in /tests/cypress (…
a9a80f9 
…#1750)
@machikoyasuda
fix: do not show scrollbar when not necessary (#1752)
b3fc711
@dependabot
chore(deps-dev): bump django from 4.2.5 to 4.2.6
54bdb1b 
Bumps [django](https://github.com/django/django) from 4.2.5 to 4.2.6.
- [Commits](django/django@4.2.5...4.2.6)

---
updated-dependencies:
- dependency-name: django
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <support@github.com>
@machikoyasuda
chore(deps-dev): bump django from 4.2.5 to 4.2.6 (#1753)
ef11391
@github-actions github-actions bot added migrations [auto] Review for potential model changes/needed data migrations updates actions Related to GitHub Actions workflows i18n Copy: Language files or Django i18n framework tests Related to automated testing (unit, UI, integration, etc.) back-end Django views, sessions, middleware, models, migrations etc. and removed back-end Django views, sessions, middleware, models, migrations etc. i18n Copy: Language files or Django i18n framework docker Application container, devcontainer, Compose, etc. actions Related to GitHub Actions workflows migrations [auto] Review for potential model changes/needed data migrations updates labels Nov 16, 2023
@github-actions GitHub Actions
Copy link

Coverage report

Note

Coverage evolution disabled because this PR targets a different branch
than the default branch, for which coverage data is not available.

The coverage rate is 90.87%.
The branch rate is 85%.

80% of new lines are covered.

Diff Coverage details (click to unfold)

benefits/eligibility/forms.py

75% of new lines are covered (80% of the complete file).
Missing lines: 165

benefits/settings.py

100% of new lines are covered (83.58% of the complete file).

@thekaveman thekaveman added i18n Copy: Language files or Django i18n framework docker Application container, devcontainer, Compose, etc. labels Nov 16, 2023
@machikoyasuda
Copy link
Member

@indexing is working on one copy edit ticket for SBMTD Mobility Pass. He said the ticket will be done by 3:30. And I can work on it quickly after that. If possible, would like to get that ticket into dev before we deploy to test.

@thekaveman @angela-tran

@thekaveman thekaveman marked this pull request as draft November 16, 2023 23:10
@machikoyasuda
Copy link
Member

Copy fixes are in to this branch 👍

@thekaveman thekaveman marked this pull request as ready for review November 17, 2023 16:51
@thekaveman thekaveman merged commit edd35c4 into test Nov 20, 2023
15 checks passed
@github-actions github-actions bot added the deployment [auto] Pull requests that deploy to test or prod label Nov 20, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@angela-tran angela-tran angela-tran approved these changes

Assignees
No one assigned
Labels
back-end Django views, sessions, middleware, models, migrations etc. deployment [auto] Pull requests that deploy to test or prod docker Application container, devcontainer, Compose, etc. front-end HTML/CSS/JavaScript and Django templates i18n Copy: Language files or Django i18n framework tests Related to automated testing (unit, UI, integration, etc.)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@thekaveman @machikoyasuda @angela-tran @indexing