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

docs: Update the contributing guidelines link. #197

Merged
merged 1 commit into from
Apr 20, 2023

Conversation

feanil
Copy link
Contributor

@feanil feanil commented Apr 20, 2023

We're moving towards a single set of guidelines org-wide.

We're moving towards a single set of guidelines org-wide.
@feanil feanil force-pushed the feanil/update_contributing_link branch from f54a07c to 660d78d Compare April 20, 2023 18:37
@feanil feanil requested a review from a team April 20, 2023 18:38
@pshiu pshiu merged commit fc1bfaf into master Apr 20, 2023
@pshiu pshiu deleted the feanil/update_contributing_link branch April 20, 2023 18:56
grmartin added a commit that referenced this pull request Aug 29, 2023
* Update standard workflow files. (#195)

* docs: Update the contributing guidelines link. (#197)

We're moving towards a single set of guidelines org-wide.

* chore: updated Python requirements (REV-2693) (#198)

* fix: load config using yaml.safe_load(), not yaml.load() (#202)

Attempt to address exception:

    Traceback (most recent call last):
      File "/edx/app/ecomworker/venvs/ecomworker/bin/celery", line 8, in <module>
        sys.exit(main())
      File "/edx/app/ecomworker/venvs/ecomworker/lib/python3.8/site-packages/celery/__main__.py", line 16, in main
        _main()
      File "/edx/app/ecomworker/venvs/ecomworker/lib/python3.8/site-packages/celery/bin/celery.py", line 322, in main
        cmd.execute_from_commandline(argv)
      File "/edx/app/ecomworker/venvs/ecomworker/lib/python3.8/site-packages/celery/bin/celery.py", line 499, in execute_from_commandline
        super(CeleryCommand, self).execute_from_commandline(argv)))
      File "/edx/app/ecomworker/venvs/ecomworker/lib/python3.8/site-packages/celery/bin/base.py", line 305, in execute_from_commandline
        return self.handle_argv(self.prog_name, argv[1:])
    [...]
      File "/edx/app/ecomworker/venvs/ecomworker/lib/python3.8/site-packages/celery/app/base.py", line 920, in _finalize_pending_conf
        conf = self._conf = self._load_config()
      File "/edx/app/ecomworker/venvs/ecomworker/lib/python3.8/site-packages/celery/app/base.py", line 930, in _load_config
        self.loader.config_from_object(self._config_source)
    [...]
      File "/edx/app/ecomworker/ecomworker/ecommerce_worker/configuration/production.py", line 18, in <module>
        config_from_yaml = yaml.load(f)
    TypeError: load() missing 1 required positional argument: 'Loader'

Docs: https://pyyaml.org/wiki/PyYAMLDocumentation#loading-yaml

* feat: update pypa's publish branch away from master

* chore: added CODEOWNERS file (#199)

Co-authored-by: Muhammad Zubair <syedzubairtahir12@gmail.com>

---------

Co-authored-by: Feanil Patel <feanil@tcril.org>
Co-authored-by: Shahroz Ahmad <97090106+ishahroz@users.noreply.github.com>
Co-authored-by: Phillip Shiu <pshiu@edx.org>
Co-authored-by: Hamzah Ullah <hamzahullah@yahoo.com>
Co-authored-by: Muhammad Zubair <syedzubairtahir12@gmail.com>
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.

2 participants