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

v1.1.1 #174

Merged
merged 17 commits into from
Apr 6, 2023
Merged

v1.1.1 #174

merged 17 commits into from
Apr 6, 2023

Conversation

vincerubinetti
Copy link
Collaborator

@vincerubinetti vincerubinetti commented Mar 24, 2023

See changelog for high level, user-facing changes.

  • docker python install changes. pin watchdog version, turn off unneeded caching.
  • docker entrypoint changes. run cite process immediately, ensure no simultaneous runs.
  • first time setup fixes. don't parse config.yaml as yaml, instead customize with string replacement to preserve key order and comments.
  • add title and link fallbacks to citation component
  • fix start/end markers for blog post excerpts, and remove special characters from hidden search attribute
  • change member page from col layout to float
  • add multi-line option to regex_scan liquid plugin (to support start/end marker bug above)
  • add new regex_strip liquid plugin to strip all special characters from string
  • card, code, portrait, post excerpt css fixes
  • fix testbed image links
  • (re)include user pr template
  • fix versioning workflow to correctly parse and split changelog
  • add show-title and show-subtitle site config options
  • include site subtitle in description meta tag

- docker python install changes. pin watchdog version, turn off unneeded caching.
- docker entrypoint changes. run cite process immediately, ensure no simultaneous runs.
- first time setup fixes. don't parse config.yaml as yaml, instead customize with string replacement to preserve key order and comments.
- add title and link fallbacks to citation component
- fix start/end markers for blog post excerpts, and remove special characters from hidden search attribute
- change member page from col layout to float
- add multi-line option to regex_scan liquid plugin (to support start/end marker bug above)
- code, portrait, post excerpt css
- fix testbed image links
- fix card component css
- fix versioning workflow to correctly parse and split changelog
- (re)include user pr template
@netlify
Copy link

netlify bot commented Mar 24, 2023

Deploy Preview for lab-website-template ready!

Name Link
🔨 Latest commit 3563ca5
🔍 Latest deploy log https://app.netlify.com/sites/lab-website-template/deploys/642f280a1f585700084d620d
😎 Deploy Preview https://deploy-preview-174--lab-website-template.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@github-actions
Copy link

github-actions bot commented Mar 24, 2023

PR Preview Action v1.3.0
Preview removed because the pull request was closed.
2023-04-06 22:43 UTC

Copy link
Collaborator

@d33bs d33bs left a comment

Choose a reason for hiding this comment

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

Nice work! I left a few minor comments and suggestions. Overall I thought it looked good!

An additional note:

  • In the deploy preview I noticed on the homepage, under "Our Projects" the link for "browse our projects" results in a page not found.

.docker/Dockerfile Show resolved Hide resolved
.github/workflows/first-time-setup.yaml Show resolved Hide resolved
.github/workflows/first-time-setup.yaml Outdated Show resolved Hide resolved
.github/workflows/versioning.yaml Show resolved Hide resolved
_plugins/regex.rb Show resolved Hide resolved
.github/user_pull_request_template.md Show resolved Hide resolved
@vincerubinetti
Copy link
Collaborator Author

Thanks for the review @d33bs , please take a look at my notes and recent commits.

Copy link
Contributor

@falquaddoomi falquaddoomi left a comment

Choose a reason for hiding this comment

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

Seems like @d33bs has done a pretty thorough review already. I made one comment, but otherwise it looks good to me.

.docker/entrypoint.sh Outdated Show resolved Hide resolved
.docker/Dockerfile Show resolved Hide resolved
.github/workflows/versioning.yaml Show resolved Hide resolved
_plugins/regex.rb Show resolved Hide resolved
@vincerubinetti vincerubinetti merged commit 7435920 into main Apr 6, 2023
@vincerubinetti vincerubinetti deleted the v1.1.1 branch April 6, 2023 22:42
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.

Python cite scripts enhancements Separate title from subtitle in config show-text
3 participants