Skip to content

bubble up warning if a user isn't compatible with upcoming v2 - #505

Merged
oscarlevin merged 2 commits into
mainfrom
try-out-refactor
Jul 28, 2023
Merged

oscarlevin merged 2 commits into
mainfrom
try-out-refactor

Conversation

@StevenClontz

Copy link
Copy Markdown
Member

I believe this is ready for the field in a 1.7.3 release.

@StevenClontz
StevenClontz requested a review from oscarlevin July 28, 2023 21:38
@oscarlevin

Copy link
Copy Markdown
Member

This looks great. I added a report of success when it works, both to buzz-market the 2.0 release, and so we can encourage everyone to run pretext for their project using 1.7.3 before they upgrade to 2.0, to see if they are ready.

@StevenClontz

Copy link
Copy Markdown
Member Author

I think adding stuff to support is duplicative as users will we the reports from main.

Maybe report success in support, and report issues in main?

@oscarlevin

Copy link
Copy Markdown
Member

My thinking for adding the report to support is that it would make it easier for us to see what the problems are when they copy the results of support to the google group (they might not naturally include the part at the top that says to post on support)

@oscarlevin
oscarlevin merged commit d22bd81 into main Jul 28, 2023
@oscarlevin
oscarlevin deleted the try-out-refactor branch July 28, 2023 22:58
oscarlevin added a commit that referenced this pull request Aug 1, 2023
* fix zip_templates so it doesn't recreated duplicate devcont (#497)

* Fix: Run tests only on a PR; do this on any branch. (#495)

* Fix: Run tests only on a PR; do this on any branch.

* Fix: Per discussion, also run testing on any pushes to the main branch.

* Fix: Update major versions of older packages. (#499)

* Fix: Update major versions of older packages.

See #496.

* Update: poetry.lock based on project file changes.

* Fix: Make mapping file platform-independent. (#504)

* bubble up warning if a user isn't compatible with upcoming v2 (#505)

* bubble up warning if a user isn't compatible with upcoming v2

* add success messages, error report to support

---------

Co-authored-by: Oscar Levin <oscar.levin@unco.edu>

* bump version to next patch level for nightly releases

* revert to previous pdfCropMargins version (#509)

* revert to previous pdfCropMargins version

* update action to trigger docker

* bump version to next patch level for nightly releases

* Update deploy-stable.yml

Quick fix to get docker to update

---------

Co-authored-by: Oscar Levin <oscar.levin@unco.edu>
Co-authored-by: Steven Clontz <steven.clontz@gmail.com>
Co-authored-by: GitHub Actions Bot <>
oscarlevin added a commit that referenced this pull request Aug 5, 2023
* fix zip_templates so it doesn't recreated duplicate devcont (#497)

* Fix: Run tests only on a PR; do this on any branch. (#495)

* Fix: Run tests only on a PR; do this on any branch.

* Fix: Per discussion, also run testing on any pushes to the main branch.

* Fix: Update major versions of older packages. (#499)

* Fix: Update major versions of older packages.

See #496.

* Update: poetry.lock based on project file changes.

* Fix: Make mapping file platform-independent. (#504)

* bubble up warning if a user isn't compatible with upcoming v2 (#505)

* bubble up warning if a user isn't compatible with upcoming v2

* add success messages, error report to support

---------

Co-authored-by: Oscar Levin <oscar.levin@unco.edu>

* bump version to next patch level for nightly releases

* revert to previous pdfCropMargins version (#509)

* revert to previous pdfCropMargins version

* update action to trigger docker

* bump version to next patch level for nightly releases

* Update deploy-stable.yml

Quick fix to get docker to update

* reconfigure playwright install process (#518)

* bump version to next patch level for nightly releases

---------

Co-authored-by: Bryan A. Jones <bjones@ece.msstate.edu>
Co-authored-by: Steven Clontz <steven.clontz@gmail.com>
Co-authored-by: GitHub Actions Bot <>
oscarlevin added a commit that referenced this pull request Aug 12, 2023
* start converting to refactored project; fix logging

* finish fixing logging

* make ptx_version optional, use legacy if not 2

* reimplement pretext support

* Fix: Make project.py the new __init__.py. (#500)

Add in new output_dir/output_file structure.

* Clean: Rename output_file to output_filename.

* Fix: Basic renames for new project class.

* Fix: More renames for new project class.

* get  working

* build working in default case

* integrate webwork-reps and output-dir checking

* Add: Support for server URLs. (#511)

Fix: Correct output-dir/output-filename tests, add more.

* Main merge (#512)

* fix zip_templates so it doesn't recreated duplicate devcont (#497)

* Fix: Run tests only on a PR; do this on any branch. (#495)

* Fix: Run tests only on a PR; do this on any branch.

* Fix: Per discussion, also run testing on any pushes to the main branch.

* Fix: Update major versions of older packages. (#499)

* Fix: Update major versions of older packages.

See #496.

* Update: poetry.lock based on project file changes.

* Fix: Make mapping file platform-independent. (#504)

* bubble up warning if a user isn't compatible with upcoming v2 (#505)

* bubble up warning if a user isn't compatible with upcoming v2

* add success messages, error report to support

---------

Co-authored-by: Oscar Levin <oscar.levin@unco.edu>

* bump version to next patch level for nightly releases

* revert to previous pdfCropMargins version (#509)

* revert to previous pdfCropMargins version

* update action to trigger docker

* bump version to next patch level for nightly releases

* Update deploy-stable.yml

Quick fix to get docker to update

---------

Co-authored-by: Oscar Levin <oscar.levin@unco.edu>
Co-authored-by: Steven Clontz <steven.clontz@gmail.com>
Co-authored-by: GitHub Actions Bot <>

* clean up pretext build (#506)

* clean up pretext build

* reenable codechat

* add zip option to html build

* incorporate suggestions

* add remaining build targets, except custom

* add custom build format

* fix ensure_webwork_reps

* remove pre-2.0 warning message (#514)

* Set up generate for refactor (#508)

* get generate working

* a few minor improvements

* Complete merge

* remove old build code

* add check for finding assets to be built

* refactor asset hash function

* generate almost working

* complete refactor of generate

* fix interactive/preview dir bug

* fix flake error

* Fix: mypy errors. (#515)

Co-authored-by: Oscar Levin <oscar.levin@unco.edu>
Just removed some conflicting changes that were no longer needed after generate refactor.

* Add: The Runestone target. (#516)

* Add: The Runestone target.

* Fix: The WeBWorK target also supports compression.

* Merge latest changes from main (#519)

* fix zip_templates so it doesn't recreated duplicate devcont (#497)

* Fix: Run tests only on a PR; do this on any branch. (#495)

* Fix: Run tests only on a PR; do this on any branch.

* Fix: Per discussion, also run testing on any pushes to the main branch.

* Fix: Update major versions of older packages. (#499)

* Fix: Update major versions of older packages.

See #496.

* Update: poetry.lock based on project file changes.

* Fix: Make mapping file platform-independent. (#504)

* bubble up warning if a user isn't compatible with upcoming v2 (#505)

* bubble up warning if a user isn't compatible with upcoming v2

* add success messages, error report to support

---------

Co-authored-by: Oscar Levin <oscar.levin@unco.edu>

* bump version to next patch level for nightly releases

* revert to previous pdfCropMargins version (#509)

* revert to previous pdfCropMargins version

* update action to trigger docker

* bump version to next patch level for nightly releases

* Update deploy-stable.yml

Quick fix to get docker to update

* reconfigure playwright install process (#518)

* bump version to next patch level for nightly releases

---------

Co-authored-by: Bryan A. Jones <bjones@ece.msstate.edu>
Co-authored-by: Steven Clontz <steven.clontz@gmail.com>
Co-authored-by: GitHub Actions Bot <>

* Refactor `view` command (#520)

* update view commands before actual view call

* port oldinit's view command

* fix typos

* start adding persistent server logic

* implement reusable server

* fix flake errors

* remove unneded module

* improve keyboard interrupt

* improve checking for running server; reporting of url

* Refactor deploy (#522)

* wire up deploy with refactor

* clean up

* Change: Instead of a Runestone format, provide a platform="runestone" attribute. (#527)

* Update: Template project files to v2 format. (#528)

* update default to server for asymptote

* Fix errors with view

* Improvements to view server checking (#529)

* Improvements to view server checking

* remove obsolete view functions

* Gen (#530)

* remove project overlay

* clean up generate code

* Fix issue with xmlid persisting through generate

* Docs for a Runestone target (#526)

* Add: Runestone elements for the demo template.

* Add: Runestone elements for the article template.

* Remove: Runestone elements from the article template.

See #526 (comment).

* Add: Runestone elements for the book template.

* Fix: Update language based on Rob's feedback.

* Fix: Update demo template as well.

* Fix: Correct indent, avoid hyphens.

* Misc fixes (#533)

* Fix: Black/Flake8/mypy errors (except for oldinit.py).

* Fix: Delete, following fac5e44.

* Fix: Move default values from Target.server to Project.server; fix syntax.

* Fix: Ignore project/oldinit.py in mypy.

* Fix: Update to newer mypy.

* Fix: Avoid syntax that requires Python 3.9.

* fix bug with non-existant assets being saved in hash

* fix qrcode typo

* remove debugging hack

* Add: Missing target from default template. (#535)

* Add: Missing target from default template.

* Fix: Correct tests for subset.

* Get test_view working (#537)

* clean up old code

* update dependencies

* clean up old code

* update dependencies

* remove depricated  test

* fix custom_webwork_server test

* Get serve test working (#538)

* Fix: Correct handling of output-filename to prepend the output-dir. (#540)

* Fix: Correct handling of output-filename to prepend the output-dir.

* Fix: Flake8.

* update core commit and version

* fix elaborate build test (#541)

* fix elaborate build test

* add playwright install for previews

* Fix view tests with connection errors

* flake8

* Add asymptote method (#542)

* add executables to gitignore

* remove bad merge

---------

Co-authored-by: Bryan A. Jones <bjones@ece.msstate.edu>
Co-authored-by: Steven Clontz <steven.clontz@gmail.com>
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.

2 participants