Skip to content

9.1 Deployment#212

Open
BHoMBot wants to merge 6 commits intomainfrom
develop
Open

9.1 Deployment#212
BHoMBot wants to merge 6 commits intomainfrom
develop

Conversation

@BHoMBot
Copy link
Copy Markdown
Member

@BHoMBot BHoMBot commented Mar 23, 2026

File_Toolkit

Compliance

@BHoMBot BHoMBot added the type:feature New capability or enhancement label Mar 23, 2026
@peterjamesnugent peterjamesnugent self-requested a review March 31, 2026 17:26
Copy link
Copy Markdown
Member

@peterjamesnugent peterjamesnugent left a comment

Choose a reason for hiding this comment

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

Checked using BHoM_9.1.260327.beta.0 with test procedures and scripts.

I couldn't get the Excel file to work, but I believe that's because of the file not any changes made to the adapter.

Copy link
Copy Markdown
Member

@peterjamesnugent peterjamesnugent left a comment

Choose a reason for hiding this comment

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

Happy to approve after the code and documentation compliance fixes given only add descriptions.

The one code change (change of class to match filename) is in the testing project.

@peterjamesnugent
Copy link
Copy Markdown
Member

@BHoMBot check compliance
@BHoMBot check required

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci bot commented Apr 1, 2026

@peterjamesnugent to confirm, the following actions are now queued:

  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check branch-compliance
  • check dataset-compliance
  • check copyright-compliance
  • check code-compliance
  • check documentation-compliance
  • check project-compliance
  • check core
  • check null-handling
  • check serialisation
  • check versioning
  • check installer

There are 2 requests in the queue ahead of you.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci bot commented Apr 1, 2026

The check beta-code-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci bot commented Apr 1, 2026

The check beta-documentation-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

@bhombot-ci
Copy link
Copy Markdown

bhombot-ci bot commented Apr 1, 2026

The check beta-project-compliance has already been run previously and recorded as a successful check. This check has not been run again at this time.

@michaelhoehn michaelhoehn self-assigned this Apr 1, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New capability or enhancement

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants