Skip to content

Fix some attributes whitespaces that were removed#217

Merged
thet merged 1 commit into
masterfrom
ale/216/class_multiline
May 31, 2026
Merged

Fix some attributes whitespaces that were removed#217
thet merged 1 commit into
masterfrom
ale/216/class_multiline

Conversation

@ale-rt
Copy link
Copy Markdown
Member

@ale-rt ale-rt commented May 31, 2026

This was caused by a BeautifulSoup configuration default that caused some attributes to be stored as a list of strings. The PR forces all attributes to be stored as string.

Refs. #216

This was caused by a BeautifulSoup configuration default that caused
some attributes to be stored as a list of strings.
The PR forces all attributes to be stored as string.

Refs. #216
@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage is 96.733%ale/216/class_multiline into master. No base build found for master.

Copy link
Copy Markdown
Member

@thet thet left a comment

Choose a reason for hiding this comment

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

LGTM and makes #216 obsolete as this is the much better solution.

@thet thet merged commit 16adb44 into master May 31, 2026
7 checks passed
@thet thet deleted the ale/216/class_multiline branch May 31, 2026 16:45
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.

3 participants