Skip to content

Improved automation tooling #405

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

Merged
merged 4 commits into from
Apr 20, 2025
Merged

Conversation

CBenghi
Copy link
Contributor

@CBenghi CBenghi commented Apr 15, 2025

Hello,

I've closed all issues open on the IDS Audit Tool repository and published a new version.

Further commits in this PR address:

  • fixing naming of the test cases to remove duplicate test names under the same type
  • improved .markdownlint.json file to help notify naming clashes in the tests.
  • fixed datatypes markdown to correct availability of the IFCSTRIPPEDOPTIONAL class to the right IFC schema versions.

@atomczak, would you have a look at this please?

CBenghi added 4 commits April 11, 2025 21:51
removed markdown warnings and more explicit title in IDS
IFCSTRIPPEDOPTIONAL was mistakenly marked valid for Ifc4,
but it should not be, since it is a new entity in Ifc4x3.
@CBenghi CBenghi requested a review from atomczak April 15, 2025 16:54
@CBenghi CBenghi merged commit efc587a into buildingSMART:development Apr 20, 2025
1 check passed
@CBenghi
Copy link
Contributor Author

CBenghi commented Apr 20, 2025

Apologies @atomczak ,

I had a remote misconfigured to buildingSmart instead of my own repo, which made this PR merge automatically as I pushed a new commit.

Feel free to revert if you find issues in the code.

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.

1 participant