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

Stop the document from jumping to the hash when a token is clicked. #352

Closed
wants to merge 2 commits into from

Conversation

jverkoey
Copy link
Collaborator

No description provided.

@jverkoey
Copy link
Collaborator Author

I'm not sure what the best way to stage the integration-specs repo is. It seems that I'd have to land the change in the other repo first in order to reference the commit here. Is that correct?

I'm wondering if instead it's possible to temporarily set the submodule ref to point to my own repo where the relevant integration-specs branch is...

@pcantrell
Copy link
Collaborator

@jverkoey Yes, you have to first create a commit in integration-specs before committing the updated reference in jazzy. You also have to make sure that integration-specs is pushed before Travis sees your update to jazzy.

@jpsim
Copy link
Collaborator

jpsim commented Nov 11, 2015

Ok, I just merged realm/jazzy-integration-specs#7. Could you please update this PR to point the integration specs submodule to that commit?

Change looks good otherwise.

* None.
* Expanding a token no longer causes the document to 'jump' to the hash.
[Jeff Verkoeyen](https://github.com/jverkoey)
[#352](https://github.com/realm/jazzy/issues/352)
Copy link
Collaborator

Choose a reason for hiding this comment

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

FYI we only include links to issues, not PRs

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Ah good to know - will fix.

@jpsim
Copy link
Collaborator

jpsim commented Nov 16, 2015

Taking this over. Thanks @jverkoey.

@jpsim jpsim closed this Nov 16, 2015
jpsim added a commit to realm/jazzy-integration-specs that referenced this pull request Nov 16, 2015
@jverkoey
Copy link
Collaborator Author

Ah thank you! I got blocked on updating the specs and then life took hold :p is there any way to more easily generate the specs?

@segiddins
Copy link
Collaborator

There's a rake task that rebuilds them

@jpsim
Copy link
Collaborator

jpsim commented Nov 16, 2015

is there any way to more easily generate the specs?

unfortunately not. you have push access to jazzy-integration-specs, so I suggest you follow these steps when updates are necessary:

git checkout master
git pull
git checkout -
git rebase master
bundle install
bundle exec rake rebuild_integration_fixtures
cd spec/integration_specs
git checkout -b $jazzy_branch_name
git commit -a -m "update for $jazzy_branch_name"
git push
cd ../../
git commit -a -m "update integration specs"
git push

@jverkoey
Copy link
Collaborator Author

Noted :) what's the rationale for having the specs as a separate repo?

@jpsim
Copy link
Collaborator

jpsim commented Nov 16, 2015

They're quite large.

@jverkoey
Copy link
Collaborator Author

Ah, makes sense. Thanks again!

@jpsim
Copy link
Collaborator

jpsim commented Nov 17, 2015

Actually, they're really not that large... I'll make a PR merging them back into jazzy proper so we can discuss the tradeoffs

@segiddins
Copy link
Collaborator

@jpsim please make the PR from a fork

@pigeondotdev pigeondotdev modified the milestone: The Past Nov 22, 2016
pitfield added a commit to codewinsdotcom/PostgresClientKit that referenced this pull request Apr 5, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants