Skip to content

Conversation

@Tom-Kingstone
Copy link
Contributor

NOTE: Depends on

Issues addressed by this PR

Closes #

Test files

Changelog

Additional comments

@Tom-Kingstone Tom-Kingstone self-assigned this Jan 29, 2026
@Tom-Kingstone Tom-Kingstone added the type:bug Error or unexpected behaviour label Jan 29, 2026
@jamesramsden-bh
Copy link
Contributor

jamesramsden-bh commented Jan 29, 2026

@BHoMBot check required

@bhombot-ci
Copy link

bhombot-ci bot commented Jan 29, 2026

@jamesramsden-bh to confirm, the following actions are now queued:

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

There are 35 requests in the queue ahead of you.

@bhombot-ci
Copy link

bhombot-ci bot commented Jan 29, 2026

@jamesramsden-bh to confirm, the following actions are now queued:

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

There are 5 requests in the queue ahead of you.

@jamesramsden-bh
Copy link
Contributor

@BHoMBot check copyright-compliance

@bhombot-ci
Copy link

bhombot-ci bot commented Jan 29, 2026

@jamesramsden-bh to confirm, the following actions are now queued:

  • check copyright-compliance

Copy link
Contributor

@jamesramsden-bh jamesramsden-bh 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 merge

@jamesramsden-bh
Copy link
Contributor

@BHoMBot check ready-to-merge

@bhombot-ci
Copy link

bhombot-ci bot commented Jan 29, 2026

@jamesramsden-bh to confirm, the following actions are now queued:

  • check ready-to-merge

@Tom-Kingstone
Copy link
Contributor Author

@BHoMBot check copyright-compliance

@bhombot-ci
Copy link

bhombot-ci bot commented Jan 30, 2026

@Tom-Kingstone to confirm, the following actions are now queued:

  • check copyright-compliance

There are 91 requests in the queue ahead of you.

@Tom-Kingstone
Copy link
Contributor Author

@BHoMBot check ready-to-merge

@bhombot-ci
Copy link

bhombot-ci bot commented Jan 30, 2026

@Tom-Kingstone to confirm, the following actions are now queued:

  • check ready-to-merge

There are 91 requests in the queue ahead of you.

Copy link
Contributor

@jamesramsden-bh jamesramsden-bh left a comment

Choose a reason for hiding this comment

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

ran tests.bat and reviewed code changes, all good

@jamesramsden-bh
Copy link
Contributor

@BHoMBot check ready-to-merge

@bhombot-ci
Copy link

bhombot-ci bot commented Feb 2, 2026

@jamesramsden-bh to confirm, the following actions are now queued:

  • check ready-to-merge

There are 20 requests in the queue ahead of you.

@bhombot-ci
Copy link

bhombot-ci bot commented Feb 2, 2026

@Tom-Kingstone just to let you know, I have provided a check-ready-to-merge result to this Pull Request as it was detected to be linked to other Pull Requests in a series. The comment which triggered this check came from @jamesramsden-bh on LadybugTools_Toolkit

@bhombot-ci
Copy link

bhombot-ci bot commented Feb 2, 2026

The check ready-to-merge has already been run previously and recorded as a successful check. This check has not been run again at this time.

@jamesramsden-bh jamesramsden-bh merged commit 9dc9eb6 into develop Feb 2, 2026
10 checks passed
@jamesramsden-bh jamesramsden-bh deleted the LadybugTools_Toolkit-#178-FixFromCsvIndex branch February 2, 2026 11:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:bug Error or unexpected behaviour

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants