Skip to content

Testing Babel#15

Draft
SriHV wants to merge 2 commits into
mainfrom
testing-babel
Draft

Testing Babel#15
SriHV wants to merge 2 commits into
mainfrom
testing-babel

Conversation

@SriHV

@SriHV SriHV commented Jan 28, 2025

Copy link
Copy Markdown
Contributor

What is the context of this PR?

Raising this PR to test the PR that Andrew raised ONSdigital/design-system#3477
Created a new test page for this

How to review this PR

Follow the readme and run the command poetry run flask run navigate to http://127.0.0.1:5000/test.
See that with our previous code we get like
image

@SriHV SriHV changed the title add new changes Testing Babel Jan 28, 2025
Comment thread babel.cfg
@@ -0,0 +1,3 @@

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Do we need this empty line?

@rmccar

rmccar commented Jan 30, 2025

Copy link
Copy Markdown
Contributor

We could do with a bit more of an explanation of the changes in this PR and a bit more detail on how to test. I understand what needs to be done but only because of previous knowledge. If we were to come back to this in the future to work out what changes were made and why it wouldn't be obvious.
Also we will need to add tests around this and rename the "test" file to something more descriptive

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