[go: nahoru, domu]

Skip to content
This repository has been archived by the owner on Jul 6, 2021. It is now read-only.

Markdown api docs #150

Merged
merged 11 commits into from
Mar 31, 2017
Merged

Markdown api docs #150

merged 11 commits into from
Mar 31, 2017

Conversation

leobalter
Copy link
Member
@leobalter leobalter commented Mar 30, 2017

I'm currently using Jekyll to generate the files because of the simplicity and it's easy to use anything else later.

Everything is minimal and can have more work.

We need a few things:

  • We might wanna set gh-pages as the main branch
  • cancel the publishing hooks.
  • set dns to point to the generated pages

@leobalter leobalter self-assigned this Mar 30, 2017
@leobalter
Copy link
Member Author

@trentmwillis
Copy link
Member

I'm super excited for this!

Also, looks like this will move us away from the jquery-wordpress setup (which I am excited for), does this mean we will be able to do some design updates? I've gotten feedback several times recently that the visual appearance of the documentation site is a put-off (because it looks outdated).

@leobalter
Copy link
Member Author

does this mean we will be able to do some design updates?

Yes! In anyway, I'll start with the most simple implementation. I'm currently using some CSS I borrowed from http://opendesignkit.org/ and I'll try to get the pages done and loading. As soon as we have it, we can improve the docs more and more.

I've gotten feedback several times recently that the visual appearance of the documentation site is a put-off (because it looks outdated).

The docs will look crude now, but open for any improvement :)

@leobalter
Copy link
Member Author

The md files for api are ready. I'll create index files for folders and start applying some style.

@leobalter leobalter changed the title [WIP] Markdown api docs Markdown api docs Mar 30, 2017
@leobalter leobalter deployed to github-pages March 30, 2017 21:43 Active
@leobalter leobalter deployed to github-pages March 30, 2017 21:53 Active
@leobalter
Copy link
Member Author
leobalter commented Mar 30, 2017

This is basically ready.

(moving the todo to the main post)

cc @gnarf on the api.qunitjs.com, can you help me, please?

@leobalter
Copy link
Member Author

to test for now: run a local jekyll serve

@aulvi
Copy link
aulvi commented Mar 30, 2017

@leobalter How can I help?

@leobalter
Copy link
Member Author

@aulvi I'm sorry for the extra flag here. @gnarf might help me tomorrow but I will need to point api.qunitjs.com to the generated gh-pages here when this is released.

@aulvi
Copy link
aulvi commented Mar 30, 2017

@leobalter No worries! Feel free to ping me direct via email tomorrow if @gnarf is unavailable, I should be wfh and am happy to help.

this can be removed later when we don't need the prefix anymore
@leobalter leobalter deployed to github-pages March 30, 2017 22:23 Active
@leobalter
Copy link
Member Author

For now: http://qunitjs.github.io/api

Copy link
@platinumazure platinumazure left a comment

Choose a reason for hiding this comment

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

Just some minor changes.

QUnit/only.md Outdated
}
});

test( "say", function( assert ) {

Choose a reason for hiding this comment

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

Would prefer this say QUnit.test unless a destructure is shown at the top of this sample.

Foo.prototype.bar = 'prototype';

var foo = new Foo( 1, "2", [] );
var bar = new Foo( "1", 2, {} );

Choose a reason for hiding this comment

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

Inconsistent indentation


### `QUnit.config.current` (object)

This object isn't actually a configuration property, but is listed here anyway, as its exported through `QUnit.config`. This gives you access to some QUnit internals at runtime. See below for an example.

Choose a reason for hiding this comment

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

s/as its exported/as it's exported

I assume this is a long-standing typo rather than anything introduced in this PR, but all the same...


Defines the HTML content to use in the fixture container which is reset at the start of each test.

By default QUnit will use whatever the starting content of `#quint-fixture` is as the fixture reset. If you do not want the fixture to be reset in between tests, set the value to `null`.

Choose a reason for hiding this comment

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

s/quint-fixture/qunit-fixture

@leobalter leobalter deployed to github-pages March 30, 2017 23:03 Active
@leobalter
Copy link
Member Author

Thanks @platinumazure, it should be fixed now.

Copy link
Member
@gibson042 gibson042 left a comment

Choose a reason for hiding this comment

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

WOW. 👏 I don't know if I'll have a chance for a deep dive, but I skimmed every file and this looks GREAT. The only question I have is whether you'd consider going even farther and moving it all into a /docs directory in the primary repository, which would allow QUnit PRs to form complete units encompassing both code and documentation changes. The work is already done here, so at this point it's just an organizational question—do we want a separate repository for documentation?

@leobalter
Copy link
Member Author

@gibson042 that's something I considered, and another reason to have md+jekyll as a +1 combo on GitHub.

We'll need to figure out how to have it in sync with the published docs, thou. That's gonna be easy anyway.

@leobalter leobalter merged commit 3a9ba6b into master Mar 31, 2017
leobalter added a commit that referenced this pull request Mar 31, 2017
@leobalter leobalter deleted the md-docs branch March 31, 2017 19:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

None yet

5 participants