Navigation Menu

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

Fix #1671 #1673

Merged
merged 4 commits into from May 2, 2015
Merged

Fix #1671 #1673

merged 4 commits into from May 2, 2015

Conversation

ralsina
Copy link
Member

@ralsina ralsina commented May 1, 2015

Fix for #1671:

  1. Avoid circular dependencies when using post-list in a post
  2. Make post.text() work anytime. It goes a bit against design principles, but I feel it's worth it because it makes the Post object more predictable.
  3. This adds a "magic dependency marker" that rst directives can use which marks the document as basically unstable, making it never be uptodate. That means it will always be rebuilt.

As a refinement, this could be improved so the rebuild is only triggered, in this case, when the timeline changes.

@Kwpolska Kwpolska changed the title Fix 1671 Fix #1671 May 1, 2015
ralsina added a commit that referenced this pull request May 2, 2015
@ralsina ralsina merged commit c1672d6 into master May 2, 2015
@ralsina ralsina deleted the fix-1671 branch May 2, 2015 02:39
@ralsina
Copy link
Member Author

ralsina commented May 2, 2015

It does work.

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.

None yet

1 participant