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

docs: Add conventions for bindings #75

Closed
wants to merge 4 commits into from

Conversation

iancooper
Copy link
Collaborator

Description
This establishes a set of conventions to use when designing bindings. The goal is that bindings should be consistent, such that they can be easily interpreted by tools. 'See #62` for an outline of this problem.

The conventions are listed as a series of items. The intent here is that items could be added (or retired) by the community as we discover better ways to work.

I don't expect that everyone will agree with these conventions, but I do believe that the points here are worthy of community agreement on a convention. I don't believe that this list is complete, but I do believe it is a "good start" for discussion around the conventions we need.

Related issue(s)
'Resolves #62`

@iancooper iancooper changed the title Add conventions for bindings docs: Add conventions for bindings Jul 14, 2021

## Effective Bindings
### **Item 1** Use the Extensions Format
Prefer use of the [Bindings Extensions](https://github.com/asyncapi/extensions-catalog) format when defining a binding over raw JSON Schema.
Copy link
Collaborator Author

@iancooper iancooper Jul 15, 2021

Choose a reason for hiding this comment

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

@fmvilas This is still experimental. I'd not recommend this yet. Maybe we should just advise using Specification Extensions: https://github.com/asyncapi/spec/blob/master/spec/asyncapi.md#specificationExtensions.

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 think it would be useful to adopt something that allowed us to provide a consistent format for a binding, other than just JSON, particularly around tooling.

@iancooper
Copy link
Collaborator Author

As a note, re-created as a fork sync tool I was using deleted the original PR

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴
It will be closed in 60 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.
Thank you for your contributions ❤️

@github-actions github-actions bot added the stale label Sep 14, 2021
@fmvilas fmvilas removed the stale label Sep 14, 2021
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Jan 13, 2022
@ericsampson
Copy link

bump

@github-actions github-actions bot removed the stale label Jan 14, 2022
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label May 15, 2022
@iancooper
Copy link
Collaborator Author

I guess the question here is how we move this forward?

@github-actions github-actions bot removed the stale label May 16, 2022
@fmvilas
Copy link
Member

fmvilas commented May 17, 2022

@iancooper that's a tricky question. What's missing here in your opinion?

@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added stale and removed stale labels Sep 15, 2022
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Jan 15, 2023
@ericsampson
Copy link

bump @iancooper is the project still working towards taking the guidance that you've provided? I kind of lost track

@github-actions github-actions bot removed the stale label Jan 19, 2023
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label May 20, 2023
@fmvilas
Copy link
Member

fmvilas commented May 30, 2023

@iancooper do you need something to move it forward?

@github-actions github-actions bot removed the stale label May 31, 2023
@github-actions
Copy link

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Sep 29, 2023
@asyncapi-bot
Copy link
Contributor

asyncapi-bot commented Oct 30, 2023

We require all PRs to follow Conventional Commits specification.
More details 👇🏼

 The subject "Add conventions for bindings" found in the pull request title "docs: Add conventions for bindings" should start with a lowercase character.

@github-actions github-actions bot removed the stale label Oct 31, 2023
@smoya smoya mentioned this pull request Nov 6, 2023
@smoya
Copy link
Member

smoya commented Nov 6, 2023

@iancooper is #211 supposed to be replacing this PR?

Copy link

github-actions bot commented Mar 6, 2024

This pull request has been automatically marked as stale because it has not had recent activity 😴

It will be closed in 120 days if no further activity occurs. To unstale this pull request, add a comment with detailed explanation.

There can be many reasons why some specific pull request has no activity. The most probable cause is lack of time, not lack of interest. AsyncAPI Initiative is a Linux Foundation project not owned by a single for-profit company. It is a community-driven initiative ruled under open governance model.

Let us figure out together how to push this pull request forward. Connect with us through one of many communication channels we established here.

Thank you for your patience ❤️

@github-actions github-actions bot added the stale label Mar 6, 2024
@smoya
Copy link
Member

smoya commented Mar 11, 2024

ping @iancooper

@iancooper
Copy link
Collaborator Author

I think this is still a useful idea. I think some events have also superseded it and it might need to catch up to the "state of play". I am going to close the PR, but sit down for a chat with @dpwdec and some of the others and contemplate what a new version of this would look like.

@iancooper iancooper closed this Mar 11, 2024
Copy link
Member

smoya commented Mar 11, 2024

Sounds like a really good plan. Thanks for the answer and for retaking this @iancooper !

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

Successfully merging this pull request may close these issues.

Establish conventions on where we define configuration for a protocol using bindings for consistency
5 participants