Skip to content
This repository has been archived by the owner on Oct 8, 2024. It is now read-only.

ci: add .deepsource.toml file for activating DeepSource #3

Closed
wants to merge 163 commits into from

Conversation

donvargax
Copy link

ℹ️ DeepSource has created this pull-request through [email protected] to activate DeepSource analysis on this repository.

This PR adds a .deepsource.toml file that will configure the following analyzers on every PR, commit, and subsequent merges to the default branch: JavaScript, Python, Shell, SQL, Terraform, Test coverage.

DeepSource will start the initial analysis the moment this PR is merged to the default branch. The results can be seen on the dashboard here: https://app.deepsource.com/gh/correlation-one/edx-ora2/. There is no additional action needed.

leangseu-edx and others added 30 commits September 28, 2022 13:54
It's an implicit dependency of xblock-sdk and we don't use it directly
here so make that more clear while we're updating the package to the new
name on PyPI
It's not clear why these constraints point to this PR which was merged 4
years ago so remove them and see if we run into any issues when running
tests.

Leaving these here is causing an issue where make upgrade can't upgrade
anymore because newer versions of `tox` have a dependency on `pluggy` at
a higher version than what `pytest==4.5.0` is compatible with.  This is
what led me to look at these constraints in the first place.
Remove constraints on older versions of zipp and mock.
In one of the tests the value of the ID is a mock.  In newer versions of
mock, the value won't get auto coerced to a number.  Also, it doesn't
make sense to coerce the UUID as a number so updated the logging to
output the UUID as a string in general.
Newer versions of tox no longer support support `whitelist_externals`.
We're meant to use `allowlist_externals` instead.
We now work with bleach version 6.0.0 which is not backwards compatible
older versions of bleach.  So we make that explicit in `base.in` which
feeds the `install_requires` for this library.
* chore: Updating Python Requirements

* fix: use node v16 in ci
The default nodejs version is 19.5 on the github ci workflow and that is causing a issue with OpenSSL mismatch. Pin the nodejs version to v16 on github workflow environment

* fix: lint amnesty on use-dict-literal
The pylint upgrade to 2.16.2 resulted in a new lint check with use-dict-literal. Fix by setting anmesty on them

---------

Co-authored-by: Simon Chen <[email protected]>
The .github/workflows/self-assign-issue.yml workflow is missing or needs an update to stay in
sync with the current standard for this workflow as defined in the
`.github` repo of the `openedx` GitHub org.
…yml`.

The .github/workflows/add-remove-label-on-comment.yml workflow is missing or needs an update to stay in
sync with the current standard for this workflow as defined in the
`.github` repo of the `openedx` GitHub org.
…d.yml`.

The .github/workflows/add-depr-ticket-to-depr-board.yml workflow is missing or needs an update to stay in
sync with the current standard for this workflow as defined in the
`.github` repo of the `openedx` GitHub org.
* Move translation directory from `openassessment/locale` to `openassessment/conf/locale`
* Create a symbolic link from `openassessment/locale` to `openassessment/conf/locale`
* Fix the translation command `make extract_translations` to use `django-admin` from inside `openassessment` directory instead of adding too many `--ignore`

Refs: FC-0012 OEP-58
* style: remove lint-amnesty for dict literals

* chore: bump version numbers to 5.0.1
* fix: make adding image or file resize correctly

chore: update the logic using trigger

chore: update logic again

chore: trigger resize event manually

* chore: bump version
nsprenkle and others added 29 commits December 6, 2023 16:35
* fix: always return a response when requesting peer

* refactor: remove unused get_peer endpoint

* fix: remove unused import

* fix: hide deleted files from submission

* chore: bump ORA to version 6.0.11
* fix: add step param to assess view

* style: remove unused import

* test: typo

* test: fix test failures

* chore: version
* fix: return empoty list for files rather than none

* fixup! fix: return empoty list for files rather than none

* feat: add max file size to file config

* chore: version

* style: quality
* feat: disable new ORA experience for leaderboards

* chore: bump ORA to 6.0.14
* feat: disable new ORA experience for LaTeX previews

* chore: bump ORA to 6.0.15
…2142)

* fix: fix serializing of null assessments in score

* chore: bump ORA to 6.0.16
* feat: re-enable ORAs with reordered steps

* chore: bump ORA to 6.0.17
…x#2099)

* chore: Update to the new version of paragon in the new scope.

Part of openedx/axim-engineering#23

This replaces the `@edx/paragon` packag to point to the `paragon` package at
the `openedx` scope(`@openedx/paragon`). Imports have been updated to use the
same locations in the new package.

* fix: Update webpack configs to transpile openedx packages.

This babel-loader config is pulled from [email protected]

We had to do this because frontend-build is currently too far behind and
can't easily be updated to a newer version without upgrading webpack and
many other libraries.

By making this small change we unblock the paragon upgrade to the new
scope and latest version.

* chore: Static asset compilation.

* build: Add an nvmrc file to be clear about the node version.

This is now best practice in repos with JS code so that it's easy to be
using the right version of node.  This repo is still on node 16 because
edx-platform is still on node 16
* feat: add function traces to BFF endpoints

* chore: bump ORA to 6.0.18
* feat: updates "has reached" check to account for skipped steps

* feat: get next incomplete step

Since new ORA UI uses "serial" instead of "parallel" progression,
need to correctly progress to peer when skipped.

 Adds check for next incomplete step and uses for progress instead of naive pull from workflow status

* chore: bump ORA to 6.0.19

* test: add test for skipped step

* test: update some test scenarios to make testing easier
* fix: also check "graded" on workflow when advancing

* chore: bump ORA to 6.0.21

* style: more consistent styling in workflow API file
* fix: allow feedback only criteria in assessment

* chore: bump ORA to 6.0.22
* chore: remove disable on assessment out of order

* chore: update test

* chore: update html tag

* chore: version bump
* chore: fix studio not reload correctly

chore: update linting

* chore: version bump
* fix: make sure we're assessing the right submission

* chore: version bump
* fix: skip check when uuid not passed

* chore: version
* feat: remove disabled LaTex for ora mfe

* chore: remove LaTex test

* chore: version bump
* feat: remove duplicate workflow lookups

Previously, anything that requested a workflow would cause a workflow
update and refresh. This changes that logic to:
1. Only init workflow API once through load of ORA block (reloaded each
time the page / ORA MFE load).
2. On load, update workflow.
3. Calls for workflow info reference cached workflow.
4. Try to update workflow when none found.

Note we can still  manually refresh workflow info with either
get_workflow_info or update_workflow_status.

* refactor: call the API version of get_workflow_info

Within the new MFE calls, we need to consistently leverage the API version of workflow calls to take advantage of the cached workflow state.

* docs: typo fixes and docstring updates

* feat: remove no-longer-needed function_trace

* fix: cache workflow on update_workflow_status

* fix: remove unnecessary update in tests

* style: more consistent return statements

* test: refactor mocking to fix tests

* chore: bump ORA to 6.0.28
* chore: update loading screen

* chore: add support for studio and preview

* chore: build static

* chore: update translation

* chore: update preview logic

* chore: version bump
* test: reproduce issue

* fix: allow blank type

* chore: version
* fix: block submitting or assessing on closed steps

* test: add tests for blocking submit/assess when step closed

* fix: block loading of page data for closed step with 400 error

* refactor: more consistent step data accessor names

* chore: bump ORA to 6.0.31
* fix: add empty rubric validation
* chore: extract translations
* build: bump new version
ci: add .deepsource.toml file for activating DeepSource
@donvargax donvargax closed this Mar 12, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.