Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

Raml 0.8/1.0 files from amf/2.0.0 #129

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

Conversation

BlockLucas
Copy link

This contains a few extra cases from amf/2.0.0 branch

  • 5 RAML 1.0
  • 63 RAML 0.8

Copy link
Contributor

@postatum postatum left a comment

Choose a reason for hiding this comment

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

I'm not sure RAML 0.8 examples are still necessary. cc @jstoiko
If they are, please see comments.

@@ -0,0 +1,28 @@
#%RAML 0.8
Copy link
Contributor

Choose a reason for hiding this comment

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

These examples(tests/raml-0.8/Methods/query-parameter-example/) seem like a duplicate of tests/raml-0.8/Methods/query-parameter-example-2/ to me. If so, please remove one of these folders.

#%RAML 0.8

# Objective: The following example illustrates authentication using OAuth 2.0 and OAuth 1.0.
# Expected result: no error message
Copy link
Contributor

Choose a reason for hiding this comment

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

Why does it expect no error but is called invalid-?

@@ -0,0 +1,31 @@
#%RAML 0.8
Copy link
Contributor

Choose a reason for hiding this comment

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

Name of the file is too generic.

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

Successfully merging this pull request may close these issues.

2 participants