Skip to content
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

Add mailing list attribute to project schema #749

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

Raghavg18
Copy link
Contributor

Resolves #701

Copy link
Collaborator

@AbhayTopno AbhayTopno left a comment

Choose a reason for hiding this comment

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

@Raghavg18 add these changes and it should work

@@ -54,6 +54,8 @@ def test_positive(project_schema):
"license-invalid-value.yaml",
"'INVALID-LICENSE-VALUE' is not one of ['AGPL-3.0', 'Apache-2.0', 'BSD-2-Clause', 'BSD-3-Clause', 'CC-BY-4.0', 'CC-BY-SA-4.0', 'CC0-1.0', 'EUPL-1.2', 'GPL-2.0', 'GPL-3.0', 'LGPL-2.1', 'LGPL-3.0', 'MIT', 'MPL-2.0', 'OTHER']",
),
("mailing_list-invalid.yaml", "'invalid-email' is not a valid email format"),
Copy link
Collaborator

Choose a reason for hiding this comment

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

Replace this line of code with this ("mailing_list-none.yaml", "None is not of type 'string'"),

Copy link
Collaborator

@AbhayTopno AbhayTopno Feb 5, 2025

Choose a reason for hiding this comment

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

replace this file name with mailing_list-none.yaml

Copy link
Collaborator

Choose a reason for hiding this comment

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

replace the file name with mailing_list-empty.yaml

name: Leader Name 1
- github: leader-name-2
name: Leader Name 2
level: 2
Copy link
Collaborator

Choose a reason for hiding this comment

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

after this line add mailing_list: ""

name: Leader Name 1
- github: leader-name-2
name: Leader Name 2
level: 2
Copy link
Collaborator

Choose a reason for hiding this comment

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

after this line add mailing_list:

@AbhayTopno
Copy link
Collaborator

AbhayTopno commented Feb 5, 2025

And also add test case in schema/tests/data/project/positive

Copy link
Collaborator

@arkid15r arkid15r left a comment

Choose a reason for hiding this comment

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

Could you remove unrelated changes and make sure tests work before requesting review?

@arkid15r arkid15r marked this pull request as draft February 6, 2025 18:46
@github-actions github-actions bot removed the backend label Feb 7, 2025
@Raghavg18
Copy link
Contributor Author

The test was not running even before this changes ig @arkid15r

@arkid15r
Copy link
Collaborator

arkid15r commented Feb 8, 2025

The test was not running even before this changes ig @arkid15r

I'm not sure what you mean. In any case your PR (as any other PR) must have all checks passed, including the tests.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add mailing list attribute to project schema
3 participants