-
Notifications
You must be signed in to change notification settings - Fork 308
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
Problems with Page.source_filename
, Page.iter_source_filenames
with alternatives are enabled
#959
Closed
Commits on Dec 13, 2021
-
Configuration menu - View commit details
-
Copy full SHA for a12f349 - Browse repository at this point
Copy the full SHA a12f349View commit details -
WIP: deprecate .source_filename, cleanup .iter_source_filenames
When alts are enabled, Page.source_file was being generated based on the page.alt. This is not correct, since the alt-specific file may not exists. E.g. if there is only a `contents.lr`, for `alt="en"` `source_file` was returning the non-existent file `contents+en.lr`. In general, `.source_filename` and `.iter_source_filenames` should return only extant source filenames. Dependency tracking for the blog pages of the example site is not working. They get rebuilt every build. This has something to do with the `Siblings` virtual source object for the blog pages not listing the correct source filenames due to the issue with Page.source_file described above.
Configuration menu - View commit details
-
Copy full SHA for 3296213 - Browse repository at this point
Copy the full SHA 3296213View commit details -
Configuration menu - View commit details
-
Copy full SHA for a4880fb - Browse repository at this point
Copy the full SHA a4880fbView commit details -
Configuration menu - View commit details
-
Copy full SHA for 8ab126c - Browse repository at this point
Copy the full SHA 8ab126cView commit details -
Configuration menu - View commit details
-
Copy full SHA for 9ed30c5 - Browse repository at this point
Copy the full SHA 9ed30c5View commit details -
Configuration menu - View commit details
-
Copy full SHA for c6c3e17 - Browse repository at this point
Copy the full SHA c6c3e17View commit details -
fixup! WIP: deprecate .source_filename, cleanup .iter_source_filenames
Refactor to add _source_filenames in lieu of _source_alts system field.
Configuration menu - View commit details
-
Copy full SHA for 1766ed4 - Browse repository at this point
Copy the full SHA 1766ed4View commit details -
Configuration menu - View commit details
-
Copy full SHA for d14a045 - Browse repository at this point
Copy the full SHA d14a045View commit details -
Hash return value of Siblings.get_checksum. General code cleanup.
There's no point in storing the huge long string containing multiple file names and hashes in the build state db.
Configuration menu - View commit details
-
Copy full SHA for 8084635 - Browse repository at this point
Copy the full SHA 8084635View commit details -
Configuration menu - View commit details
-
Copy full SHA for 69141a9 - Browse repository at this point
Copy the full SHA 69141a9View commit details
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.