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

Release/astro blog rss #13

Closed

Conversation

alexaveldanez
Copy link
Contributor

@alexaveldanez alexaveldanez commented Nov 21, 2024

Description Of Changes

These changes have been made to be able to convert the blog to use Astro.

Motivation and Context

This change is necessary to be able to convert the blog to use Astro.

These changes also allow for generating RSS feed and atom feed.

Testing

Tested in localhost by previewing the Astro blog website and being able to view the rss.xml and atom feed.xml in a Google chrome browser.

Operating Systems Testing

Windows 10 Pro

Change Types Made

  • Bug fix (non-breaking change).
  • Feature / Enhancement (non-breaking change).
  • Breaking change (fix or feature that could cause existing functionality to change).
  • Documentation changes.
  • PowerShell code changes.

Change Checklist

  • Requires a change to the documentation.
  • Documentation has been updated.
  • Tests to cover my changes, have been added.
  • All new and existing tests passed?
  • PowerShell code changes: PowerShell v3 compatibility checked?

Related Issue

ENGTASKS-4029
#12

Fixes #
#12

@st3phhays
Copy link
Member

So it's easier to track later, can you fixup the 4 commits in this PR into one commit, and add some words to the commit message?

st3phhays and others added 2 commits November 21, 2024 14:20
This upgrades the dependencies used by choco-astro to their latest
versions. These changes should not effect any functionality.
Add packages to be able to generate RSS feed and atom feed in Astro blog.
@alexaveldanez
Copy link
Contributor Author

So it's easier to track later, can you fixup the 4 commits in this PR into one commit, and add some words to the commit message?

I have merged the 4 commits into one and added a message to the commit.

@alexaveldanez
Copy link
Contributor Author

Closing in favor of PR [https://github.com//pull/14].

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