Skip to content

Update to edition-arica of the Eiffel protocol #68

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.

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

Merged
merged 3 commits into from
Feb 15, 2023

Conversation

t-persson
Copy link
Collaborator

Applicable Issues

fixes: #67

Description of the Change

Updated to the latest schemas

Alternate Designs

N/A

Benefits

N/A

Possible Drawbacks

N/A

Sign-off

Developer's Certificate of Origin 1.1

By making a contribution to this project, I certify that:

(a) The contribution was created in whole or in part by me and I
have the right to submit it under the open source license
indicated in the file; or

(b) The contribution is based upon previous work that, to the best
of my knowledge, is covered under an appropriate open source
license and I have the right under that license to submit that
work with modifications, whether created in whole or in part
by me, under the same open source license (unless I am
permitted to submit under a different license), as indicated
in the file; or

(c) The contribution was provided directly to me by some other
person who certified (a), (b) or (c) and I have not modified
it.

(d) I understand and agree that this project and the contribution
are public and that a record of the contribution (including all
personal information I submit with it, including my sign-off) is
maintained indefinitely and may be redistributed consistent with
this project or the open source license(s) involved.

Signed-off-by: Tobias Persson tobias.persson@axis.com

@t-persson t-persson requested a review from a team as a code owner January 10, 2023 09:57
@t-persson t-persson requested review from fredjn and magnusbaeck and removed request for a team January 10, 2023 09:58
@t-persson
Copy link
Collaborator Author

I'll try to break out the workflow changes and create a separate PR for those (including fixing the warnings and errors) during the day or tomorrow

Copy link
Member

@magnusbaeck magnusbaeck left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You need to update the link types table too, right? RUNTIME_ENVIRONMENT and PRECURSOR are missing (maybe more).

@@ -1,4 +1,5 @@
sphinx<2
pyscaffold==2.5.10
mistune==0.8.4
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Where and how is this dependency used?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It is used by m2r (the line below) and m2r does not set version and the project is set to read-only. This meant that m2r wanted to install a version of mistune that was not backwards compatible thus breaking m2r for us.

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I want to remove m2r when we are updating to support newer python versions

@t-persson
Copy link
Collaborator Author

Added a commit with Older gevent version is not compatible with greenlet as well as a rebase against master

@magnusbaeck
Copy link
Member

Still LGTM :-)

@t-persson t-persson merged commit 9e8669d into eiffel-community:master Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to Arica
3 participants