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

added content to adapters.md #22

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

added content to adapters.md #22

wants to merge 3 commits into from

Conversation

A1Liu
Copy link
Contributor

@A1Liu A1Liu commented Oct 18, 2018

No description provided.

@A1Liu
Copy link
Contributor Author

A1Liu commented Oct 18, 2018

So based on comments from @Bad-Science on the previous pull request, I'm thinking of the following:

  • The example code should be written in a high level language - The code should look as similar to pseudo-code as possible. The more abstract the syntax is, the easier it will be to generalize and explain concisely
  • The example code should use as few packages as possible - Using packages requires some level of explanation with what each package does, and what the syntax of each means.

Maybe the example adapter code should be in python?

@A1Liu
Copy link
Contributor Author

A1Liu commented Oct 18, 2018

We can probably finish some kind of preliminary adapter for the gallatin course search api and use it to explain the whole adapter process

@Bad-Science
Copy link
Member

@A1Liu Agreed, python is probably a good common denominator. Let's get this first adapter or so done and then we can get back to work on this.

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.

2 participants