Skip to content

Conversation

@dependabot
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Apr 26, 2021

Bumps ini from 1.3.5 to 1.3.8.

Commits
  • a2c5da8 1.3.8
  • af5c6bb Do not use Object.create(null)
  • 8b648a1 don't test where our devdeps don't even work
  • c74c8af 1.3.7
  • 024b8b5 update deps, add linting
  • 032fbaf Use Object.create(null) to avoid default object property hazards
  • 2da9039 1.3.6
  • cfea636 better git push script, before publish instead of after
  • 56d2805 do not allow invalid hazardous string as section name
  • See full diff in compare view
Maintainer changes

This version was pushed to npm by isaacs, a new releaser for ini since your current version.


Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)
  • @dependabot use these labels will set the current labels as the default for future PRs for this repo and language
  • @dependabot use these reviewers will set the current reviewers as the default for future PRs for this repo and language
  • @dependabot use these assignees will set the current assignees as the default for future PRs for this repo and language
  • @dependabot use this milestone will set the current milestone as the default for future PRs for this repo and language

You can disable automated security fix PRs for this repo from the Security Alerts page.

Bumps [ini](https://github.com/isaacs/ini) from 1.3.5 to 1.3.8.
- [Release notes](https://github.com/isaacs/ini/releases)
- [Commits](npm/ini@v1.3.5...v1.3.8)

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Apr 26, 2021
@maximecb
Copy link
Contributor

@XrXr should we merge this or not? dependabot opened 3 PRs.

I'm thinking we probably shouldn't merge it since it could change our benchmark's performance, and it might actually break the code, which would be a pain to fix?

@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 26, 2021

Dependabot tried to update this pull request, but something went wrong. We're looking into it, but in the meantime you can retry the update by commenting @dependabot rebase.

@XrXr XrXr closed this Apr 26, 2021
@dependabot @github
Copy link
Contributor Author

dependabot bot commented on behalf of github Apr 26, 2021

OK, I won't notify you again about this release, but will get in touch when a new version is available. If you'd rather skip all updates until the next major or minor version, let me know by commenting @dependabot ignore this major version or @dependabot ignore this minor version.

If you change your mind, just re-open this PR and I'll resolve any conflicts on it.

@dependabot dependabot bot deleted the dependabot/npm_and_yarn/benchmarks/railsbench/ini-1.3.8 branch April 26, 2021 23:03
byroot added a commit to byroot/yjit-bench that referenced this pull request Jan 29, 2025
THe initial snapshot was from a few years back, the application changed
a lot since then.

I had to run a bunch of migrations and had to fiddle with various sqlite3/msyql
incompatibilities. Also some route changes, but with a bit of effort I managed
to make it work again.

What is interesting is it seems lobste.rs became 2-3-x as slow in these
couple years, I haven't yet profiled to know why.

This branch:

```
itr:   time
 ruby#1: 1532ms
 ruby#2: 1013ms
 ruby#3:  910ms
 ruby#4:  852ms
 ruby#5:  851ms
 ruby#6:  901ms
 ruby#7:  857ms
 ruby#8:  886ms
 ruby#9:  875ms
```

main:

```
itr:   time
 ruby#1:  884ms
 ruby#2:  482ms
 ruby#3:  344ms
 ruby#4:  358ms
 ruby#5:  300ms
 ruby#6:  316ms
 ruby#7:  315ms
 ruby#8:  328ms
 ruby#9:  309ms
```
byroot added a commit to byroot/yjit-bench that referenced this pull request Jan 30, 2025
THe initial snapshot was from a few years back, the application changed
a lot since then.

I had to run a bunch of migrations and had to fiddle with various sqlite3/msyql
incompatibilities. Also some route changes, but with a bit of effort I managed
to make it work again.

What is interesting is it seems lobste.rs became 2-3-x as slow in these
couple years, I haven't yet profiled to know why.

This branch:

```
itr:   time
 ruby#1: 1532ms
 ruby#2: 1013ms
 ruby#3:  910ms
 ruby#4:  852ms
 ruby#5:  851ms
 ruby#6:  901ms
 ruby#7:  857ms
 ruby#8:  886ms
 ruby#9:  875ms
```

main:

```
itr:   time
 ruby#1:  884ms
 ruby#2:  482ms
 ruby#3:  344ms
 ruby#4:  358ms
 ruby#5:  300ms
 ruby#6:  316ms
 ruby#7:  315ms
 ruby#8:  328ms
 ruby#9:  309ms
```
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants