Skip to content

Conversation

@usarid
Copy link
Contributor

@usarid usarid commented Jul 14, 2017

No description provided.

Copy link
Contributor

@fehguy fehguy left a comment

Choose a reason for hiding this comment

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

I think this looks great. As noted, we will have to clean up some of the docs we're linking to, but this PR is 👍 from my POV.


The OpenAPI Specification is a community driven, open project hosted by the Linux Foundation. The OAI encourages participation from individuals and companies alike. If you want to participate in the evolution of the OpenAPI Specification, consider taking the following actions:
The current process for development of the OpenAPI Specification is described in
[Development Guidelines](DEVELOPMENT.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

It's good to leave this, but we will need to update this doc

Copy link
Contributor Author

Choose a reason for hiding this comment

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

That should be a different PR - who can take that?

The OpenAPI Specification is a community driven, open project hosted by the Linux Foundation. The OAI encourages participation from individuals and companies alike. If you want to participate in the evolution of the OpenAPI Specification, consider taking the following actions:
The current process for development of the OpenAPI Specification is described in
[Development Guidelines](DEVELOPMENT.md).
Development of the next version of the OpenAPI Specification is guided by the [OAI Technical Contributors Board](CONTRIBUTORS.md). This group of committers bring their API expertise, incorporate feedback from the community, and expand the group of committers as appropriate. All development activity on the future specification will be performed as features and merged into this branch. Upon release of the future specification, this branch will be merged to master.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Needs to get updated with TDC language

Choose a reason for hiding this comment

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

I suggest replacing "OAI Technical Contributors Board" with simply "TDC Contributors" - it's important to note that the OAI or its official members have no direct influence or mandate on the work/direction of the TDC - the suggested wording could be interpreted differently.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Updated in the gdoc

The OpenAPI Specification is a community driven, open specification within the [Open API Initiative](https://www.openapis.org/), a Linux Foundation Collaborative Project.

The current, released version of the OpenAPI Specification is 2.0, through donation of the Swagger Specification to the OAI by SmartBear Software. If you are interested in the release specification, see the [master branch](https://github.com/OAI/OpenAPI-Specification/blob/master/README.md) of this project.
The goal of the OpenAPI Specification is to define a standard, language-agnostic interface description for REST APIs which allows both humans and computers to discover and understand the capabilities of the service without access to source code, documentation, or through network traffic inspection. When properly defined via OpenAPI, a consumer can understand and interact with the remote service with a minimal amount of implementation logic. Similar to what interface descriptions have done for lower-level programming, the OpenAPI Specification removes the guesswork in calling the service.

Choose a reason for hiding this comment

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

Suggest: "The OpenAPI Specification defines ..."

Choose a reason for hiding this comment

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

Suggest: "without access to source code, additional documentation," (the OpenAPI definition is documentation)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, incorporated into the gdoc

The goal of the OpenAPI Specification is to define a standard, language-agnostic interface description for REST APIs which allows both humans and computers to discover and understand the capabilities of the service without access to source code, documentation, or through network traffic inspection. When properly defined via OpenAPI, a consumer can understand and interact with the remote service with a minimal amount of implementation logic. Similar to what interface descriptions have done for lower-level programming, the OpenAPI Specification removes the guesswork in calling the service.

Development of the next version of the OpenAPI Specification is guided by the [OAI Technical Contributors Board](https://github.com/OAI/OpenAPI-Specification/blob/OpenAPI.next/CONTRIBUTORS.md). This group of committers bring their API expertise, incorporate feedback from the community, and expand the group of committers as appropriate. All development activity on the future specification will be performed as features and merged into this branch. Upon release of the OpenAPI Specification, this branch will be merged to master.
Use cases for machine-readable API interfaces include interactive documentation, code generation for documentation, client, and server, as well as automated test cases. OpenAPI descriptions describe APIs via YAML or JSON documents that adhere to the OpenAPI Specification documented in this repository. These documents can either be produced and served statically, or be generated dynamically from your application.

Choose a reason for hiding this comment

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

Suggest: "Use cases for machine-readable API interfaces include interactive documentation; code generation for documentation, client, and server; automated test cases."

Choose a reason for hiding this comment

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

Remove "in this repository" - that is, the document is the spec (the fact that it is in a GigHub repository is unimportant)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, incorporated into the gdoc

Use cases for machine-readable API interfaces include interactive documentation, code generation for documentation, client, and server, as well as automated test cases. OpenAPI descriptions describe APIs via YAML or JSON documents that adhere to the OpenAPI Specification documented in this repository. These documents can either be produced and served statically, or be generated dynamically from your application.

The current process for development of the OpenAPI Specification is described in [Development Guidelines](https://github.com/OAI/OpenAPI-Specification/blob/OpenAPI.next/DEVELOPMENT.md).
Without going into a long history of interfaces to Web Services, this is not the first attempt to do so. We can learn from CORBA, WSDL and WADL. These specifications had good intentions but were limited by proprietary vendor-specific implementations, being bound to a specific programming language, and goals which were too open-ended. In the end, they failed to gain traction.

Choose a reason for hiding this comment

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

Suggestion: replace the first sentence with "The OpenAPI Specification design was informed by lessons learned from CORBA, WSDL, WADL, and other historical efforts in defining web services. These other specifications ..."

(I'm not sure one can claim CORBA or WSDL did not gain traction - they did, but were not easy to use. Also, this para seems more related to REST vs other web service architecture, so feels misplaced.)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I'm going to omit the paragraph entirely. I don't believe it belongs in the readme of the spec, maybe more in commentary somewhere else.


## Current Version - 3.0

The current version of the OpenAPI specification is 3.0 - and you can find it [here](versions/3.0.md).

Choose a reason for hiding this comment

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

Suggest: The current version of the OpenAPI specification is [OpenAPI Specification 3.0](versions/3.0.md).

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, incorporated into the gdoc.


The current version of the OpenAPI specification is 3.0 - and you can find it [here](versions/3.0.md).

### [Previous Versions](versions/2.0.md)

Choose a reason for hiding this comment

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

defer to the link to 2.0 to the paragraph, not the header.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, incorporated into the gdoc.


### [Previous Versions](versions/2.0.md)

This repository also contains the OpenAPI Specification 2.0, which is identical to the Swagger 2.0 specification,

Choose a reason for hiding this comment

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

[OpenAPI Specification 2.0](versions/2.0)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, incorporated into the gdoc.

Choose a reason for hiding this comment

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

Oops, sorry.... that should be [OpenAPI Specification 2.0](versions/2.0.md) (i.e. the .md extension is required)

The OpenAPI Specification is a community driven, open project hosted by the Linux Foundation. The OAI encourages participation from individuals and companies alike. If you want to participate in the evolution of the OpenAPI Specification, consider taking the following actions:
The current process for development of the OpenAPI Specification is described in
[Development Guidelines](DEVELOPMENT.md).
Development of the next version of the OpenAPI Specification is guided by the [OAI Technical Contributors Board](CONTRIBUTORS.md). This group of committers bring their API expertise, incorporate feedback from the community, and expand the group of committers as appropriate. All development activity on the future specification will be performed as features and merged into this branch. Upon release of the future specification, this branch will be merged to master.

Choose a reason for hiding this comment

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

[Technical Developer Community](https://www.openapis.org/participate/how-to-contribute/governance#TDC), not OAI Technical Contributors Board

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks, incorporated a slight modification of this in the gdoc.

@olensmar
Copy link

@usarid i'd be happy to help make the suggested changes - but your branch doesn't seem to be available...

@usarid
Copy link
Contributor Author

usarid commented Jul 19, 2017

Thanks @olensmar I've been too slammed to look into what happened here, but looking at it now.

@usarid
Copy link
Contributor Author

usarid commented Jul 24, 2017

This was eventually implemented in #1281 so let's close this one.

@usarid usarid closed this Jul 24, 2017
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.

5 participants