Skip to content

Updated DataFrame vs Buffer blogpots & added new alias Methods#10

Merged
Alokxk merged 2 commits into
pharo-containers:mainfrom
Alokxk:fix-minor-naming-issues
Jul 18, 2025
Merged

Updated DataFrame vs Buffer blogpots & added new alias Methods#10
Alokxk merged 2 commits into
pharo-containers:mainfrom
Alokxk:fix-minor-naming-issues

Conversation

@Alokxk

@Alokxk Alokxk commented Jul 18, 2025

Copy link
Copy Markdown
Collaborator

Changes:

  • Added new alias methods add: & addAll: for push: & pushAll:
  • Added tests for alias
  • Modified initialisation method for buffer
  • Improved & fixed issue in DataFrame vs Buffer blopost & update the results accordingly for the benchmark comparison

@Alokxk
Alokxk merged commit 62c604e into pharo-containers:main Jul 18, 2025
12 checks passed
@github-actions

github-actions Bot commented Jul 18, 2025

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 16361173203

Details

  • 18 of 18 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+2.2%) to 98.305%

Totals Coverage Status
Change from base Build 16095540017: 2.2%
Covered Lines: 232
Relevant Lines: 236

💛 - Coveralls

3 similar comments
@github-actions

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 16361173203

Details

  • 18 of 18 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+2.2%) to 98.305%

Totals Coverage Status
Change from base Build 16095540017: 2.2%
Covered Lines: 232
Relevant Lines: 236

💛 - Coveralls

@github-actions

github-actions Bot commented Jul 18, 2025

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 16361173203

Details

  • 18 of 18 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+2.2%) to 98.305%

Totals Coverage Status
Change from base Build 16095540017: 2.2%
Covered Lines: 232
Relevant Lines: 236

💛 - Coveralls

@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 16361173203

Details

  • 18 of 18 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+2.2%) to 98.305%

Totals Coverage Status
Change from base Build 16095540017: 2.2%
Covered Lines: 232
Relevant Lines: 236

💛 - Coveralls

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