DanielVZ96
commented
9 months ago @jmbowman I made some rough counts for all warnings from the provision run. There are plenty that are not repeated, almost all of them being deprecation warnings, with a big majority being deprecated js packages.
Given their frequencies, I'd summarize them in the following issues:
-
Fix/Ignore: [Bourbon] [Deprecation]
remis deprecated and will be removed in 5.0.0. -
[From 2 external dependencies, one can be updated, the other needs to be ignored] Fix: Django now detects this configuration automatically. You can remove default_app_config.
-
Fix: Flag '...' accessed without a request, which is likely in the context of a celery task.
-
Replace Bourbon mixins with Autoprefixer
-
Fix XBlock warnings
- [xblock-google-drive] DeprecatedPackageWarning: Please use import xblock.utils.publish_event instead of xblockutils.publish_event because the 'xblock-utils' package has been deprecated and migrated to within 'xblock' package.
- edX Studio doesn't support third-party handler urls for XBlock <class 'xblock.internal.VideoBlockWithMixins'>
- [xblock, edx-platform] DeprecationWarning: XBlocks should not instantiate their own field_data store during parse_xml()
-
[no idea what repo it's from] Fix: ModelBase.objects model manager will include soft-deleted objects in an upcoming release; please use ModelBase.available_objects to continue excluding soft-deleted objects.
-
Fix: Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0
-
Fix Python deprecation warnings: collections.abc [this repo, need to update libsass to 0.15.0] , imp module [this repo, need replace boto with boto3 in a few lines], HTTPResponse.getheaders(), add_custom_parameter
-
Fix: Flags not at the start of the expression
-
Update deprecated packages in package.json
Note: I'll check if an issue is related to this repo before I create it, so the task list length may end up being less than the list above
Counts
| count | warning extract |
|---|---|
| 165 | rem is deprecated and will be removed |
| 129 | Django now detects this configuration automatically. You can remove default_app_config. |
| 120 | DeprecationWarning: Deprecated call to `pkg_resources.declare_namespace' |
| 35 | accessed without a request, which is likely in the context of a celery task. |
| 31 | is deprecated and will be removed in 5.0.0. We suggest using an automated prefixing tool, like Autoprefixer. |
| 31 | DeprecationWarning: 'etree' is deprecated. Use 'xml.etree.ElementTree' instead. |
| 30 | XBlock <class 'xblock.internal.DiscussionXBlockWithMixins'> does not contain field url_name |
| 24 | ModelBase.objects model manager will include soft-deleted objects in an upcoming release; please use ModelBase.available_objects to continue excluding soft-deleted objects. |
| 22 | transition is deprecated and will be removed in 5.0.0. We suggest using an automated prefixing tool, like Autoprefixer. |
| 18 | because the 'xblock-utils' package has been deprecated and migrated to within 'xblock' package. |
| 17 | Using / for division outside of calc() is deprecated and will be removed in Dart Sass 2.0.0 |
| 12 | DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated since Python 3.3, and in 3.10 it will stop working |
| 6 | please upgrade to graceful-fs 4 for compatibility with current and future versions of Node.js |
| 6 | Please update to minimatch 3.0.2 or higher to avoid a RegExp DoS issue |
| 6 | deprecated browserslist@1.7.7: Browserslist 2 could fail on reading Browserslist >3.0 config used in other tools. |
| 5 | the imp module is deprecated in favour of importlib; |
| 5 | edX Studio doesn't support third-party handler urls for XBlock <class 'xblock.internal.VideoBlockWithMixins'> |
| 4 | Using / for division is deprecated and will be removed in Dart Sass 2.0.0. |
| 4 | deprecated uuid@3.4.0 |
| 3 | DeprecationWarning: XBlocks should not instantiate their own field_data store during parse_xml() |
| 2 | skipping integrity check for git dependency |
| 2 | Mac users should run this from both the lms and studio shells |
| 2 | Falling back to fake video id due to Course Blocks API failure... |
| 2 | deprecated core-js@2.6.12: core-js@<3.4 is no longer maintained and not recommended for usage due to the number of issues. Because of the V8 engine whims, feature detection in old core-js versions could cause a slowdown up to 100x even if nothing is polyfilled. Please, upgrade your dependencies to the actual version of core-js. |
| 2 | Cannot inherit metadata to * block with KVS |
| 1 | Use of runtime.render_template is deprecated. Use MakoService.render_template or a JavaScript-based template instead. |
| 1 | runtime.user_id is deprecated. Use block.scope_ids.user_id or the user service instead. |
| 1 | ANTIALIAS is deprecated and will be removed in Pillow 10 |
Other warnings
Python:
- Flags not at the start of the expression.
HTTPResponse.getheaders()is deprecated inurllib3.add_custom_parameterAPI in New Relic is deprecated.- Transcript
"subs_3_yD_cEKoCk.srt.sjson"for video"5e5d9190-9985-43ac-b859-0063968f88f2"is not found.
JavaScript:
- babel-eslint is now
@babel/eslint-parser. - @babel/polyfill is deprecated.
- bfj-node4 has been replaced by
bfj. - boom, cryptiles, hoek, sntp are deprecated.
- bower has a security vulnerability.
- chokidar v2 is deprecated.
- circular-json has been succeeded by
flatted. - extract-text-webpack-plugin is deprecated.
- formatio is unmaintained.
- har-validator is no longer supported.
- hawk has moved to
@hapi/hawk. - istanbul is no longer maintained.
- jscs has merged with ESLint.
- messageformat has been renamed.
- mkdirp older versions are deprecated.
- natives is deprecated.
- npmconf has been reintegrated into npm.
- phantomjs-prebuilt is deprecated.
- querystring API is considered Legacy.
- queue-async has been renamed.
- request is deprecated.
- resolve-url is deprecated.
- samsam is deprecated.
- source-map-resolve and source-map-url are deprecated.
- svgo v0.7.2 is deprecated.
- topojson is deprecated.
- tough-cookie is deprecated.
- urix is deprecated.
- sphinxcontrib-openapi 0.8.1 is deprecated.
Agrendalath
jmbowman
devkhan
Many warnings routinely get logged when doing console operations in edx-platform. This is especially visible when provisioning a devstack installation (although the problem isn't limited to devstack). These warnings often indicate latent problems with the code that will cause headaches when attempting future upgrades, they distract developers when looking for the root cause of a problem, and some cause a disproportionate amount of log clutter because they reappear every time a management command is called. We should clean up as many of these as practical to improve the developer experience when working with the repository.
A/C: