Skip to content

Conversation

@bobholt
Copy link
Member

@bobholt bobholt commented Nov 29, 2012

Removing unused exercise files from ajax, effects, and events.

I don't find any other references to external exercises, solutions, or demos, so I believe this issue can be closed. Any that skipped my attention can be re-opened as separate issues.

@ajpiano
Copy link
Member

ajpiano commented Nov 29, 2012

Assuming you did a once-over of all the content in its present state, this should be good to go -- this may seem redundant but can you just double-confirm that for me @bobholt? Otherwise - awesome timing!

@bobholt
Copy link
Member Author

bobholt commented Nov 29, 2012

I did a SublimeText global find for 'exercise', 'demo', and 'solution' on a fresh pull. I certainly understand if we don't want to trust it 100%.

I can put squishy eyes on the site over the next day or so.

@ajpiano
Copy link
Member

ajpiano commented Nov 29, 2012

Gotcha - in that case I can merge this in (these should definitely be gone) and then reopen #137 so you can do a final "squishy" once-over, sound good?

@bobholt
Copy link
Member Author

bobholt commented Nov 29, 2012

Sounds good.

On Wed, Nov 28, 2012 at 7:56 PM, adam j. sontag notifications@github.comwrote:

Gotcha - in that case I can merge this in (these should definitely be
gone) and then reopen #137https://github.com/jquery/learn.jquery.com/issues/137so you can do a final "squishy" once-over, sound good?


Reply to this email directly or view it on GitHubhttps://github.com//pull/204#issuecomment-10830572.

ajpiano added a commit that referenced this pull request Nov 29, 2012
@ajpiano ajpiano merged commit e679609 into jquery:master Nov 29, 2012
@ajpiano
Copy link
Member

ajpiano commented Nov 29, 2012

Thanks a lot @bobholt !

arthurvr pushed a commit to arthurvr/learn.jquery.com that referenced this pull request Jan 4, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants