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

Update framework metadata information. #35

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

parfeon
Copy link

@parfeon parfeon commented May 17, 2021

Add '.pubnub.yml' file

Add .pubnub.yml file with change logs and distirbution options.

Add '.pubnub.yml' validators

Add GitHub Actions for .pubnub.yml file validation.

Add `.pubnub.yml` file with change logs and distirbution options.

build: Add '.pubnub.yml' validators

Add GitH	ub Actions for `.pubnub.yml` file validation.
@parfeon parfeon requested a review from crimsonred May 17, 2021 21:38
@parfeon parfeon self-assigned this May 17, 2021
source-repository: https://github.com/pubnub/eon-map
documentation: https://www.pubnub.com/developers/eon
distributions:
- distribution-type: source

Choose a reason for hiding this comment

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

this should be api-client

Copy link
Author

Choose a reason for hiding this comment

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

Distribution type? Usually it is package, source

Choose a reason for hiding this comment

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

Source is not there in the enum (and source is equivalent to api-client)

                    "api-client",                         "library",                         "package",                        "application",                        "framework"

Copy link
Author

@parfeon parfeon May 19, 2021

Choose a reason for hiding this comment

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

Where?

"distribution-type": {
                        "type": "string",
                        "description" : "The type of the distribution. e.g. package"
                      },

Choose a reason for hiding this comment

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

Please check artifact-type, I will double check if we need the same enum here as well

Copy link
Author

Choose a reason for hiding this comment

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

But you left comment on distribution type (or GitHub app for phone is buggy)

Choose a reason for hiding this comment

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

ya I mentioned it on distribution type, and it should use the same enums from artifact-type. (I will double check though)

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.

None yet

2 participants