Kyle McCormick

Results 65 issues of Kyle McCormick

### Context "edx-sandbox" is the environment in which user-authored edx-platform code is safely executed using the codejail library. Today, the edx-sandbox environment's requirements are [installed from edx-platform](https://github.com/openedx/edx-platform/tree/master/requirements/edx-sandbox). One of its...

code health

Originally written up by @jmbowman over here: https://github.com/edx/edx-arch-experiments/issues/177 ____________ There are a few reasons we would prefer to always install Python packages as wheels rather than source tarballs: * Faster,...

code health

### Background Overlaps with: * https://github.com/openedx/edx-platform/blob/master/docs/decisions/0017-reimplement-asset-processing.rst * https://github.com/openedx/edx-platform/issues/31895 ### Goal edx-platform's static assets can be built without installing a Python interpreter or any Python packages. Benefits: * **Better staged image...

epic

## Context [OEP-45](https://open-edx-proposals.readthedocs.io/en/latest/architectural-decisions/oep-0045-arch-ops-and-config.html) makes declarations about how Open edX services should be structured & packaged as to simplify configuration and deployment. Tutor is aligned with some of these guidelines, such...

discovery

### Background As part of https://github.com/openedx/wg-developer-experience/issues/146, we want to optimize `pip install` for edx-platform. The more packages that edx-platform requires, the longer it will take `pip install` to run, the...

Things that need updating: * Remove link to devstack * Add link to Tutor * Explain basic "bare metal" setup steps (@feanil) , including the new `npm run build` step.

DRAFT -- This is blocked by https://github.com/openedx/edx-platform/pull/34231. ## Description This reverts commit https://github.com/openedx/edx-platform/commit/0b7e27390c1769adaa09c6b1a05f6c6a8d2104bf , added in: * https://github.com/openedx/edx-platform/pull/34130 We remove SkillTaggingMixin from common.py's XBLOCK_MIXINS because: * Skill tagging did not...

Codejail has a global variable at `codejail.jail_code.COMMANDS`. It's a dictionary that maps command names (like `"python"`) to safe shell commands. An example value is: ```python {'python': {'cmdline_start': ['/edx/app/edxapp/venvs/edxapp-sandbox/bin/python', '-E', '-B'],...

code health

(I'm not sure whether this is an issue with edx-django-utils, with [django-user-tasks](https://github.com/openedx/django-user-tasks), or both...) Adding `@set_code_owner_attribute` and `@shared_task` to the same function causes errors when that function is an instance...

Per https://openedx.atlassian.net/wiki/spaces/COMM/pages/3438280709/Adding+Codecov, I think this is the right thing to do. @dcoa , could you review this?