text:
chore: enforce PR title format with PRLint Reloaded

Description
Enforce Conventional Commit format of PR title using PRLint Reloaded
Step 1: Allow access to the codeigniter4 org then to this repo for the app (I have no rights to do so)
Step 2: Merge this PR to let the configuration kick in
For the regex reference: https://regex101.com/r/Yh1iT9/1
Checklist:

[x] Securely signed commits
[ ] Component(s) with PHPDoc blocks, only if necessary or adds value
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[ ] Conforms to style guide

I'm not sure what others will say, but I believe we were thinking of going in the opposite direction in terms of tightening PR requirements.
Potentially speaking... every commit would be checked, or only the most recent?

I think the idea of this PR is great, but it will definitely be an additional hurdle especially for beginners to contribute.
However, many PRs consist of numerous commits with unhelpful commit messages.
So my two cents are:

If PRs are merged into the respective branches "as is", so pulling through every single commit, this approach will greatly increase readability and traceability, but keep new people from contributing to the project.
If PRs are merged by "squash and merge", the goal would rather be to have the commit message of the squashed commit linted. E.g., make sure that the PR title is useful and then use that as the commit message to be merged.

This approach only checks PR titles upon creation, not commit messages involved in the PR, correct?
If so, I think this is quite useful and not a hurdle keeping beginners from contributing.

Ok, maybe I didn't fully understand it. If only the PR title is checked, I would be fine with it.

@lonnieezell thanks for the approval! can you install the Github app to the org so this can be merged?

@lonnieezell thanks for the approval! can you install the Github app to the org so this can be merged?

Sure. How would I do this?

Install it from here: https://github.com/marketplace/prlintreloaded

@paulbalandan Should be installed now for the codeigniter4 org.

Rebased to test the app. And all passed 🎉

authors: lonnieezell michalsn paulbalandan sfadschm


text:
Meta image logic fix

Fixed condition:

if try_one is None:

which is always False, because
 try_one = self.get_meta_content(doc, 'meta[property="og:image"]')

never returns None

Changed the condition order. Icon image is least relevant for top_img

This is a fantastic fix @mercuree. Also great work on the unit tests 👍 💯 Double checking and merging it now!

Wait, this appears to be failing the CI build - the failure is in the new unit test you added:
https://travis-ci.org/codelucas/newspaper/jobs/279784081
Weird how it passed the CI build locally in the comment though

I'll take a look

Haha problem found! Ironically the new bug was due to an optimization from your earlier PR!
Your PR here (https://github.com/codelucas/newspaper/pull/355) improved our xml parse optimization by turning regex off .. BUT this PR was based on the code before your optimization and thus does not turn on the regex api.
new code in this PR:
 elems = self.parser.getElementsByTag(doc, **link_img_src_kwargs)

I'll submit a PR later tonight before deploying a new version that re-enables regex operations (or performs two queries)

source: gharchive/pull-request

authors: codelucas mercuree

repo: codelucas/newspaper

license: MIT

license_type: permissive

license_source: github-api


text:
[Markdown] header not highlight correctly

Here is my markdown text
## highlight

<a>

## should highlight

Expected behavior
## should highlight should be highlight
Actual happen

not highlight as header

The problem appears to be an unescaped <a>, which causes the mode to continue treating content as HTML until a matching </a> is found.
@0b10011 Do you know if a blank line is supposed to terminate an inline HTML tag?

I think so.
Implementations seem to be torn on this, but the majority, including Gruber's markdown and commonmark.js, contain the <a> in a paragraph, which results in the browser closing the <a> before the </p>.
Gruber's markdown says "block-level HTML elements — e.g. <div>, <table>, <pre>, <p>, etc. — must be separated from surrounding content by blank lines". <a> is by default display: inline and historically could not wrap around display: block elements properly, but now it can be set to display: block. So, with <a> on it's own line separated by blank lines, one could infer that it is a block level element. But, I'd say it should just rely on the default, which is display: inline.
The CommonMark spec implies that it naively parses the HTML. That is, it will recognize open tags (and attributes), close tags, comments (<!--...-->), processing instructions (<?...?>), declarations (<!...>), and CDATA sections (<![CDATA[...]]>), and won't escape them (even non-existent tags, so long as they're valid syntax). But it says nothing about doing anything more, and the examples show opening tags without closing tags being enclosed in a paragraph. Although it doesn't have multiline examples (that I could find), I'd say that's evidence of what they intended.
So, I'd say inline HTML should be terminated by a blank line.

Agreed. Do you have time to look into fixing that in the mode?

I was hoping to get to it, but didn't end up having a chance to. I'll still try to get to it when I have time, but I'm not sure when that will be, so someone else is free to pick this up. Next time I'm able to dive into the code, if no one has taken care of this yet, I will!

source: gharchive/issue

authors: 0b10011 StevenYuysy marijnh

repo: codemirror/CodeMirror


text:
CodeMirror support Clickhouse sql mode

Hi, I was wondering if code mirror will ever have a mode for the database Clickhouse syntax highlighting or is there a way to do it with codemirror now? Thanks!

I don't know anything about this language, but no, CodeMirror will not support it out of the box, so if you want support, you'll have to figure out how to write it yourself (possibly using mode/sql/sql.js as a basis).

authors: Dennayz marijnh


text:
Highlight NULL instead of null in C/C++

CodeMirror C and C++ currently highlight the string "null", which isn't defined in C/C++. Those languages do macro "NULL", though.

Thanks. Merged!

authors: marijnh vincentwoo


text:
[clojure mode] Further clean-up.

This PR:

Highlights special forms and core symbols as keyword, following the same convention used by the Common Lisp mode and Scheme mode;
Removes indentation rules for non-core symbols to stay focused on covering more core symbols for indentation.

Removes indentation rules for non-core symbols to stay focused on covering more core symbols for indentation.

I don't quite get this—how does removing those symbols help cover more core symbols?

Oh, my bad. I should have articulated my thoughts more clearly.
My focus for this round of refactoring is on special forms and core symbols and the goal is to make syntax highlighting more consistent (by using the same style) and indentation as complete as possible (by covering the remaining core symbols). I expect this to take a little while given the number of remaining symbols in the core namespace and therefore I would like to be "laser-focued" and not distracted by other concerns (i.e., one namespace at a time).
The next step will be thinking about covering symbols from non-core namespaces. I don't know if we should yet since both the Common Lisp mode and the Scheme mode don't seem to do so. If we somehow decide to cover these namespaces, I would like to apply the same approach and do it one namespace at a time while maintaining the same level of consistency and completeness.
So that's the plan and that's why I removed those symbols for now in order to add of them back later (maybe) with better quality of coverage.

both the Common Lisp mode and the Scheme mode don't seem to do so

The CL mode has a heuristic for (see assumeBody) for when to use body-style indentation versus argument-style indentation. Maybe something like that would be appropriate here too? Though I don't know what the Clojure community's conventions look like.
I guess the number of removed symbols in this PR is small, and any regression it might cause obscure, so I'll go ahead and merge.

The CL mode has a heuristic for (see assumeBody) for when to use body-style indentation versus argument-style indentation. Maybe something like that would be appropriate here too?
Yes and that's exactly what I've been thinking about doing (instead of completely relying on a static list of symbols). The Common Lisp mode is a very good example and starting for this. I've been experimenting with tweaking it to syntax highlight Clojure code and it feels very promising. I might end up re-writing the Clojure mode based on the code of the Common Lisp mode since it is relatively easier to reason about than the current Clojure mode code.

authors: finalfantasia marijnh


text:
_regeneratorRuntime not defined when using this plugin

Using preset 'stage-0' which includes async/await.

Please add more information
I try to reproduce your problem, but stage-0 not use _regeneratorRuntime (with our plugin, and without)
I compile this code, used only stage-0 preset
async function foo() {
  return 'foo' + (await bar());
}
async function bar() {
  return 'bar';
}
foo().then(x=>console.log(x))

Without plugin
'use strict';
let foo = (() => {
  var ref = _asyncToGenerator(function* () {
    return 'foo' + (yield bar());
  });

  return function foo() {
    return ref.apply(this, arguments);
  };
})();

let bar = (() => {
  var ref = _asyncToGenerator(function* () {
    return 'bar';
  });

  return function bar() {
    return ref.apply(this, arguments);
  };
})();

function _asyncToGenerator(fn) { return function () { var gen = fn.apply(this, arguments); return new Promise(function (resolve, reject) { function step(key, arg) { try { var info = gen[key](arg); var value = info.value; } catch (error) { reject(error); return; } if (info.done) { resolve(value); } else { return Promise.resolve(value).then(function (value) { return step("next", value); }, function (err) { return step("throw", err); }); } } return step("next"); }); }; }

foo().then(x => console.log(x));

With plugin
'use strict';
let foo = (() => {
  var ref = _asyncToGenerator(_ref);

  return function foo() {
    return ref.apply(this, arguments);
  };
})();

let bar = (() => {
  var ref = _asyncToGenerator(_ref2);

  return function bar() {
    return ref.apply(this, arguments);
  };
})();

function _asyncToGenerator(fn) { return function () { var gen = fn.apply(this, arguments); return new Promise(function (resolve, reject) { function step(key, arg) { try { var info = gen[key](arg); var value = info.value; } catch (error) { reject(error); return; } if (info.done) { resolve(value); } else { return Promise.resolve(value).then(function (value) { return step("next", value); }, function (err) { return step("throw", err); }); } } return step("next"); }); }; }

function* _ref() {
  return 'foo' + (yield bar());
}

function* _ref2() {
  return 'bar';
}

foo().then(x => console.log(x));

Please, write your minimal problem code, and .babelrc(any plugins/presets except stage-0 and closure-elimination ?)
Order in babelrc - may be important also

.babelrc
{
  "env": {
    "production": {
      "presets": [ "es2015-webpack", "react", "stage-0" ],
      "plugins": [
        "closure-elimination",
        "transform-runtime",
        "transform-decorators-legacy",
        "jsx-control-statements",
        "add-module-exports",
        "transform-flow-strip-types"
      ]
    }
  }
}

I tried to move it to the bottom in the plugin list I still get.

@beckend I tried your .babelrc, but does have import _regeneratorRuntime from 'babel-runtime/regenerator'; in transformed code, and it's no problem with webpack2 build. Could you please provide code?

It's a very large codebase and I use async await everywhere...I'm sorry that I can't share it since it's closed source anyways.

I don't know what use case causing this issue, the problem no explanation, maybe you can use a simple code to reproduce this issue? :)

i have a test case that only occurs when i use as well babel-plugin-lodash:
file:

import { sortBy } from 'lodash'

async function editThing () {

}

code:
r = babel.transform(str, { presets: ['es2015', 'stage-1'], plugins: ['transform-runtime', 'closure-elimination', 'lodash']})
console.log(r.code)

result:
'use strict';

var _regenerator = require('babel-runtime/regenerator');

var _regenerator2 = _interopRequireDefault(_regenerator);

var _asyncToGenerator2 = require('babel-runtime/helpers/asyncToGenerator');

var _asyncToGenerator3 = _interopRequireDefault(_asyncToGenerator2);

var _sortBy2 = require('lodash/sortBy');

var _sortBy3 = _interopRequireDefault(_sortBy2);

var editThing = function () {
  var ref = (0, _asyncToGenerator3.default)(_ref);
  return function editThing() {
    return ref.apply(this, arguments);
  };
}();

function _interopRequireDefault(obj) { return obj && obj.__esModule ? obj : { default: obj }; }

var _marked = [_ref].map(_regeneratorRuntime.mark);

function _ref2(_context) {
  while (1) {
    switch (_context.prev = _context.next) {
      case 0:
      case 'end':
        return _context.stop();
    }
  }
}

function _ref() {
  return _regenerator2.default.wrap(_ref2, _marked[0], this);
}

_regeneratorRuntime should really be _regenerator2.default
/cc @jdalton

Which version of babel-plugin-lodash?

babel-plugin-lodash@3.1.0
babel-plugin-closure-elimination@1.0.6
babel-core@6.8.0

thanks!

Try the latest babel-plugin-lodash@3.1.4

same result :/
https://github.com/jongleberry/babel-preset-jongleberry if that's any help. i commented out this plugin in isomorphic.js and react.js

It would help if you provide what the expected output is and also file an issue in the babel-plugin-lodash repo.

@jonathanong please could you try setting "passPerPreset": true in your babelrc? Issues like this are often related to plugin ordering, so fiddling with that might be worthwhile.

@phpnode did not help
@jdalton will do so shortly!

https://github.com/lodash/babel-plugin-lodash/pull/64

Summary of the finding here. It looks like a closure-elimiation issue.

You should be able to repro this without babel-lodash-plugin using path.hub.file.addImport.

@jdalton yeah I didn't expect it to be lodash' plugin but I found it weird that the error only happened when using it. Thanks for looking into it!

No problem. Happy to help. Hit me up anytime.

Still have problems?
Please, try version 1.1.0

authors: Gvozd beckend jdalton jhen0409 jonathanong phpnode

license: mit

license_source: bigquery


text:
Get list of vertexes coordinates

Hi, I'm quite new to Leaflet and Javascript so please excuse my ignorance. First of, this is truly amazing, it looks so good and work great.
I want to get a list of coordinates all the vertexes of a polygon once drawn also just a side note, what is the layer name or are all the features placed in a group layer? thanks.

hey there
from the documentation:
// listen to when a new layer is created
map.on('pm:create', function(e) {
    e.shape; // the name of the shape being drawn (i.e. 'Circle')
    e.layer; // the leaflet layer created
});

e.layer is a regular leaflet layer and you can get all information there via the usual leaflet functions: http://leafletjs.com/reference-1.2.0.html#polyline-getlatlngs

That is fantastic, thanks so much.

authors: JacobusS codeofsumit


text:
Update to scrivito sdk

背景
Scrivito Rails SDK を 1.13.0 のアップデートする #283
調査内容
Scrivito Rails SDKはgem installで導入するものです。
今回のバージョンアップで app/assets/stylesheets/application.scss から requires scrivito しなくてもよくなりました

Further improvements
Removed CSS dependency
When setting up a Scrivito-based Rails application, it is no longer necessary to include the scrivito CSS asset in your app's stylesheets in “app/assets/stylesheets/application.css”. So, you can remove the following line from this file:
*= require "scrivito"
For compatibility reasons, this asset is still shipped with the Scrivito gem, so no action is required here when lifting your application to Scrivito 1.13.

https://www.scrivito.com/scrivito-rails-sdk-1-13-0-has-been-released-25839972428a8dd0
作業内容

Gemfileのscrivitoのバージョンを固定するようにしました
requiresしないように修正しています。
pgの version範囲を指定するようにしました

@nalabjp バージョン指定ですが scssの記述の削除もあるので指定する必要があると考えました。
その為 ~> 1.13.0 の指定をするように修正しました

PR & Review ありがとうございます! 😸 こちらもマージしちゃいますね!  🔧 💨

authors: AnaTofuZ yasulab


text:
Report broken link

The link to 'Docker and make' in Prerequisites is not working. See 'Reproducible research' session >  'Prerequisites' > '2. Docker and make are optional.'

Thanks for reporting this! Indeed this is something we used to require but we made it very optional and forgot to update that.

authors: bast primage

repo: coderefinery/reproducible-research


text:
[FEATURE] Add "Scroll-to-Top" button

Hi @codervivek5,
I would like to add a feature to scroll to top as a button on the home page.
Please assign me this issue under GSSOC'23.

Hello @codervivek5,
Please assign me this issue.

another candidate is also working on a similar to this issue so at this time I cant assign it to you.


text:
Auto Migrate redirected repos

When a repo moves, move the users:
http://www.codetriage.com/defunkt/resque =>
http://www.codetriage.com/resque/resque
Maybe give a "migrate" this repo button of some kind where it puts your request in a manual queue or opens a github issue or something.
I could have sworn this was an open issue, but I cannot find it ATM.

I looked into this recently and with the help of Github support found two ways of doing this:
(1) The non-API 'hacky' route (guess which one of us came up with this idea :smile:)
Although the API doesn't redirect on repository or user, github.com does. For example, 6to5/6to5 was recently renamed to babel/babel. Running curl -I https://api.github.com/repos/6to5/6to5 returns a 404. However, running curl -I https://github.com/6to5/6to5 returns a 301 with a Location field that points to the new location.
So, one alternative is to rescue GitHubBub::RequestError with a HEAD request to github.com, grab the new Location, massage that into an API request to the renamed repo, and update everything accordingly.
I think this would be a viable solution if there were no way to do this through the API, but Ivan from Github support showed me what may be a better alternative.
(2) The API route less travelled
All repositories and users have numerical IDs that uniquely identify them — we've just had no reason to store them...until maybe now. If we were to store them, then you could use that to ensure you get the same repository regardless of what it is named.
Take babel/babel again whose repo ID is 24560307. Knowing this, curl -i https://api.github.com/repositories/24560307 will always give us that repo even if it were renamed to Isomorphic/MonadicArrow tomorrow. Only if it were deleted (or maybe if it went private) would you run into problems.
You could use this same idea to solve the name switching bug mentioned in issue #301, too, using a call to https://api.github.com/user/:id.
Anyway, let me know your thoughts when you have time. I'd love to take a crack at solving this.
Cheers,
O-I

@O-I Awesome work. :+1: I think second approach is best.

OK. I'll look into going that route then.

:wave:
Is there any update or workaround for this? I wouldn't wanna introduce duplication but I just moved PragTob/benchee to bencheeorg/benchee and would like to know how to best proceed :D

Still needed

Fixed on #1722
Can we close this issue?

authors: O-I PragTob hbontempo-br prathamesh-sonpatki schneems

repo: codetriage/CodeTriage


text:
Allow other data types than FILE

upload method assumes the data will be a FILE, but a user of the SDK might want to upload generally anything - blob, Uint8Array, string...
Since Codex itself does not work with files, but simply with a stream of bytes, the SDK should accept other types - or at least some JS alternative to bytes

The upload method is accepting now the same type as the send method of XMLHttpRequest so Document | XMLHttpRequestBodyInit.
Please update to version 0.0.12.

repo: codex-storage/codex-js


text:
Loader

New

editor-js/translate-inline#1 👌

authors: TatianaFomina talyguryn

repo: codex-team/icons


text:
[Feature Request] Ability to specify the Heading Lvl for new day

Greetings,
First of all, great extension.
Secondly, to my request, would it be possible to add a setting to define the Heading level of a New Day?
I want to use this extension in Combination with dendronhq/dendron to improve the ability to do this. I would love the mentioned request to define the Heading from ## to ### to include it underneath a heading of level ##. Is this feasible on your end.
If you don't have the time for it, could you point me to where I would need to look to add it myself?
Friendly Regards,
flammehawk

Thanks,
to change the number of # you can change row  222 on file src/taskUtils.ts
Friendly Regards,
Luca
Il giorno dom 11 lug 2021 alle ore 12:10 flammehawk <
@.***> ha scritto:

Greetings,
First of all, great extension.
Secondly, to my request, would it be possible to add a setting to define
the Heading level of a New Day?
I want to use this extension in Combination with dendronhq/dendron
https://github.com/dendronhq/dendron to improve the ability to do this.
I would love the mentioned request to define the Heading from ## to ###
to include it underneath a heading of level ##. Is this feasible on your
end.
If you don't have the time for it, could you point me to where I would
need to look to add it myself?
Friendly Regards,
flammehawk
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/codicelq/mytime/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AR4EK36B7IBFHGCT4ZZ3GVTTXFU2TANCNFSM5AFESSZQ
.

authors: codicelq flammehawk

repo: codicelq/mytime


text:
Write a Scala program to implement binary search

Description
Write a Scala program to implement binary search
Binary search is a fast search algorithm with run-time complexity of Ο(log n). This search algorithm works on the principle of divide and conquer. For this algorithm to work properly, the data collection should be in the sorted form.
Binary search looks for a particular item by comparing the middle most item of the collection. If a match occurs, then the index of item is returned. If the middle item is greater than the item, then the item is searched in the sub-array to the left of the middle item. Otherwise, the item is searched for in the sub-array to the right of the middle item. This process continues on the sub-array as well until the size of the subarray reduces to zero.
Pseudocode
procedure binary_search
   A ← sorted array
   n ← size of array
   x ← value to be searched

   Set lowerBound = 1
   Set upperBound = n

   while x not found
      if upperBound < lowerBound
         EXIT: x does not exists.

      set midPoint = lowerBound + ( upperBound - lowerBound ) / 2

      if A[midPoint] < x
         set lowerBound = midPoint + 1

      if A[midPoint] > x
         set upperBound = midPoint - 1

      if A[midPoint] = x
         EXIT: x found at location midPoint
   end while

end procedure

Example
list = [1,2,3,4,5]
value = 4

Output : 3

How to contribute

Comment !assign to assign this issue to yourself
Fork this repository
Create a new branch
Save the solution in program/program/implement-binary-search/implementBinarySearch.scala
Commit the changes
Create a pull request

!assign

repo: codinasion/codinasion


text:
codinasion v2 :)

Why:
Codinasion v2
What's being changed:

Upgrade to Next js 13 (with app experimental feature)
Add Tailwind CSS

I just started learning Tailwind CSS but the results look good to me 😍 .
Much helpful if you could improve something 🤗

Hey, I am here 🤩

Hey @0ME9A
Any other suggestion/idea to improve it

Wait I am currently not at home so I'll give my opinion after some time 👀

Sure 🤗

Hey @harshraj8843,
It's pretty good, but you really don't want that I design a brand new Codinasion website design.😊

It's pretty good, but you really don't want that I design a brand new Codinasion website design.

It will be a great help if you will do it. We can close this PR.
I started learning Tailwind CSS so I thought to give it a try 😃

It's pretty good, but you really don't want that I design a brand new Codinasion website design.

It will be a great help if you will do it. We can close this PR.
I started learning Tailwind CSS so I thought to give it a try 😃

First of congratulation (tailwindcss) 🎉🎉🎉
And you can keep it continue as a lab because I also need some time in design, coding🧑💻

I will go ahead to approve this. It can always change later


text:
Write a Python program to find average of numbers by recursion

Description
Write a Python program to find average of numbers by recursion
Average of numbers is the sum of all numbers divided by the number of numbers. For example, the average of 1, 2, 3, 4, 5 is 3. The average of 1, 2, 3, 4, 5, 6 is 3.5.
Average of numbers can be calculated by recursion. The average of numbers is the sum of the first number and the average of the rest of the numbers. The average of the rest of the numbers is the sum of the second number and the average of the rest of the numbers. The average of the rest of the numbers is the sum of the third number and the average of the rest of the numbers. And so on.
Input  : 1 2 3 4 5
Output : 3

How to contribute

Save the solution in program/find-average-of-numbers-by-recursion.py file
Add find-average-of-numbers-by-recursion.py file in find-average-of-numbers-by-recursion folder

Hey, can I work on this? Assign me please.

Hey @swapn652
You can comment !assign on any good first issue you want to contribute to, to be auto-assigned
!assign

!assign

repo: codinasion/program


text:
Cannot override css_class_name on class instantiation

Hi,
https://github.com/codingjoe/django-select2/blob/ed1af309908d1c8c94a189209b0a40479ee95320/django_select2/forms.py#L83
The css_class_name attribute on all widgets classes inheriting Select2Mixin cannot be overridden when instantiating the class. Because there is no
def __init__(self, *args, **kwargs):
    ...
    this.css_class_name = kwargs.get("css_class_name")
    ...

in any __init__().
The only way to override css_class_name is to create a new class inheriting the desired widget: not very handy.
Is this intentional? Can we improve this?

Hi @adrien-delhorme,
Yes, that was intentional. The thought behind it was, to have a separate Python class for a different HTML element class. Is there any particular reason, why doesn't inheritance work well in your use case? I am curious. There are so many ways to do things.
Cheers!
Joe

authors: adrien-delhorme codingjoe


text:
Hi, I am a newbie, I can't use TabNine in my VSCode

I install the TabNine extension in my VSCode for python, but it's not work for me, it must be used with any setting, but the completion window still not show out, what can I do??

what's step can I verify the TabNine is worked ??

Same issue, I think this project is dead.

Same same

same. it did not show the belief of prediction as the TabNine ads. I am not sure whether it is activated.

Same thing for me

Same thing!
There is no auto-completion for Javascript, PHP nor even Python.
Its is giving all the things same as emmet.

Hi @SwastikBhattacharjee , can you please share a screenshot of your status-bar, a screenshot of the extensions section in vscode where Tabnine is installed, and the vscode console log (Help -> Toggle Developer Tools)?
Thanks

authors: SwastikBhattacharjee ZacharyWaseda dimacodota lerayflorian navixlee r3nderer

repo: codota/tabnine-vscode


text:
Scheduled Jobs

https://docs.vapor.codes/advanced/queues/
https://github.com/nginxinc/docker-nginx/blob/master/entrypoint/docker-entrypoint.sh

Chall did this

repo: cohoe/swiftarr


text:
Fix SDK Playground Export

Summary
Next 14 removed the next export command in favor of the output: export in the next.config file
How did you test your changes?
yarn build

🟡 Heimdall Review Status

Requirement
Status
More Info

Reviews

🟡
0/1

Denominator calculation

Show calculation

1 if user is bot
0

1 if user is external
0

From .codeflow.yml

1

Additional review requirements

Show calculation

Max
0

0

From CODEOWNERS

0

Global minimum
0

Max

1

1

1 if commit is unverified
0

Sum
1

authors: cb-heimdall cb-jake

repo: coinbase/coinbase-wallet-sdk


text:
Authed client can't get product ticker

The docs clearly state

AuthenticatedClient inherits all of the API methods from PublicClient, so if you're hitting both public and private API endpoints you only need to create a single client.

But if I run const tick = await authedClient.getProductTicker();, I get { "message": "NotFound" }.
The authedClient is created correctly because I can perform authedClient.buy() just fine.

Same issue using npm-installed gdax 0.4.2

authors: drootang meteorpublish

repo: coinbase/gdax-node


text:
Allow max concurrency limits configuration on the indexer

Is your feature request related to a problem? Please describe.
allowing max concurrency config will help in running index on smaller cpu machines and also free up some bitcoin rpc bandwidth for other uses.

Thanks for posting this @HelloKashif! Think this is a great "first issue" for someone looking to contribute.

authors: HelloKashif patrick-ogrady

repo: coinbase/rosetta-bitcoin


text:
[WIP] Network Status Command

Fixes #50
Motivation

Solution

Open questions

Pull Request Test Coverage Report for Build 1563

2 of 2 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.05%) to 60.185%

Totals

Change from base Build 1557:
0.05%

Covered Lines:
455

Relevant Lines:
756

💛  - Coveralls

Review Error for qiwu7 @ 2020-07-16 01:03:21 UTC
User failed mfa authentication, see go/mfa-help

repo: coinbase/rosetta-cli


text:
Fix Multiple Issues: Page Load Speed, API Key Settings, Model Selection, and Sidebar Behavior

Summary of Changes for This PR:

Fixed slow page loading:
Resolved an issue where refreshing the page under poor network conditions caused the dynamic model list to be fetched each time, leading to a white screen for over ten seconds.
Fixed API Key setting issues:
Addressed a bug where API Key settings on the page were often ineffective with providers like OpenAILike. Even after setting a new API Key, conversations continued using the old one.  This is because the update of the child component has no effect on the parent component's useEffect
Fixed invalid model selection:
Corrected issues where switching the model provider did not update the model selection. Additionally, when the server-side model list was out of sync with the client, the server would arbitrarily default to using the "claude-3-5-sonnet-latest" model for requests without informing the user, resulting in discrepancies with the model displayed in the Web UI.
Fixed sidebar annoyance:
Adjusted the behavior of the sidebar, which frequently expanded when the mouse moved near it. It now only expands when the sidebar button is clicked.

Added an expand/collapse event to the sidebar icon on the right side of the top Bolt logo. The original sidebar icon had no functionality. Now, users can manually expand or collapse the sidebar by clicking the icon, preventing the sidebar from expanding frequently when the mouse moves near it during chat input on the left side.

Fully implemented dynamic model parameter passing, eliminating the synchronization issues between front-end and back-end model parameters.

The original implementation had both the server and client maintain their own copies of the MODEL LIST, which had the following drawbacks:

The client's MODEL LIST is not sourced from the server, and when the user sets the API KEY, the client's MODEL LIST changes.
When the server pulls the MODEL LIST, it can only use the API KEY set in the environment.
The server cannot recognize new models coming from the client.

Passed API key, model, and other information through the request body, removing hidden [Model:] and [Provider:] directives, enhancing reliability.

The original API KEY, model settings, and other parameter passing were very disorganized. API keys for all providers were passed in the request body, API keys were also fetched from cookies, and hidden directives like [MODEL:] were inserted into user input. This approach significantly increased unreliability.
Now, API KEY and model parameters are uniformly passed through the request body, while the provider remains the same as before, meaning dynamic providers are not supported. The advantage of this approach is that, in the future, users can add custom MODEL lists and modify model parameters on the client side without needing to modify the server-side code or restart the server.

Fixes #532, fixes #539, fixes #551 , fixes  #451, fixes #558

I stuck with Bolt for several days and even fixed a ton of serious bugs for it.
Yet today, Cline took me away in just five minutes and showed me the truly correct path.

I no longer use bolt, so these merge conflicts will not be resolved, keep these errors intact.

cline blew my fucking mind today too.

cline blew my fucking mind today too.

Yes, cline is like a dimensional reduction strike against Bolt.


text:
Adicionar assinatura do autor no post

Adicionar uma área com foto, nome e mini biografia do autor no post.

Incluir nessa assinatura uma breve descrição do projeto Colecionando Games e os links para o Instagram e YouTube

repo: colecionando-games/blog


text:
Validation rules are getting applied for hidden fields. How can I handle that.

Based on the selection I am hiding/showing some of the form fields. But when I submit the form hidden fields validation is also triggering and I am not able to submit the form.  I have tried with ".partial()" also but still it's not working. Please let me know how we can resolve this issue.

Please send a full reproducible code example.

authors: JacobWeisenburger neumadhu

repo: colinhacks/zod


text:
each log line appearing twice

For as long as I can recall, every log line produced by delayed_job appears twice, in two different formats. I'd love to disable this (because it's just noise) but especially because I can only manipulate one of the copies.
My log config:
   # config/environments/development.rb
   ...
   # use a custom formatter for delayed jobs
   Delayed::Worker.logger = ActiveSupport::Logger.new(STDOUT)
   Delayed::Worker.logger.formatter = DelayedJobFormatter.new

   # print the output as it comes
   STDOUT.sync = true
   ...

The first copy has just the Worker host and pid and the message, e.g.:
[Worker(host:9b3bea8e62b5 pid:1)] Starting job worker

The second copy includes more information, but it clearly refers to the same event:
["INFO", 2020-01-08 14:44:05 -0500, nil, "2020-01-08T14:44:05-0500: [Worker(host:9b3bea8e62b5 pid:1)] Starting job worker"]

My custom formatter can change the second format into whatever I want, but the first format still appears.
Please help! We're using v4.1.4 of this gem on Ruby 2.6.5 with Rails 5.0.0.rc2
Thanks.
P.S. I am using lograge, but I don't think that its use is related to this problem. (When I disable lograge, I still see duplicate lines for delayed_jobs.)
P.P.S. The code of my custom formatter:
# app/models/delayed_job_formatter.rb

class DelayedJobFormatter

  def call(*data)
    # data is an Array
    "{\"level\":\"#{format_level(data)}\",\"content\":\"#{format_content(data)}\"\}\n"
  end

  protected

  def format_content(data)
    if data.fourth.match?("queue=default")
      # this is a status message
      content = data.fourth.split('(queue=default) ').second
    else
      # this is another message
      content = data.fourth.split(')] ').second
    end
    content
  end

  def format_level(data)
    # this output will be a string that is the level
    starting_level = data.first
    if starting_level == "INFO" && format_content(data).match?("failed")
      level = "ERROR"
    else
      level = starting_level
    end
    level
  end

end

I now understand why this happens, but it leads me to a different question...
Basically, by specifying a logger for Delayed::Jobs in my Rails config, I'm telling it to duplicate the logging that the gem already outputs to STDOUT.
So: new question #1: Can this output be suppressed, leaving me with ONLY my specified logger output?
That would be helpful, but since I've also learned that I can format the default output by making a plugin, at a minimum I won't have to tolerate duplicate output.

Also, adding QUIET=true to my environment variables has removed all the messages that I wasn't able to catch by using the lifecycle hooks, e.g.:
[Worker(host:13782e20caa1 pid:1)] Starting job worker
[Worker(host:13782e20caa1 pid:1)] Job ActiveJob::QueueAdapters::DelayedJobAdapter::JobWrapper (id=19) (queue=default) RUNNING
[Worker(host:13782e20caa1 pid:1)] Job ActiveJob::QueueAdapters::DelayedJobAdapter::JobWrapper (id=19) (queue=default) FAILED (8 prior attempts) with NoMethodError: undefined method `update' for nil:NilClass

Whew. I've been struggling with this myself. Adding some additional context for future folks who stumble on this page.
In case you're curious, the duplication is caused by the statement puts text unless @quiet in Delayed::Worker#say.
The default duplicate logging is still an issue and is fixed by #987 (unmerged). Although the default for the quiet option is supposed to be true, there's a bug due to the way ENV['QUIET'] is handled when you run rake jobs:work (see /lib/delayed/tasks.rb) coupled with the way the quiet option is handled when initializing new instances of Delayed::Worker (specifically @quiet = options.key?(:quiet) ? options[:quiet] : true).
And to reiterate @caljess599, you can eliminate the duplication issue in the meantime by setting ENV['QUIET'] = true.

Hi I already tried to add these lines in development.rb but I still didn't see any logging happen directly in rails application, does anyone know why?
 Delayed::Worker.logger = ActiveSupport::Logger.new(STDOUT)
 STDOUT.sync = true

The goal is to write the delayed job log in the Rails application, so that I can see the log in the pod log, not in log/delayed_job.log file. The application is hosted in openshift.


text:
Profile picture becomes distorted in member profile when browser size is decreased

Description
The user's profile picture becomes distorted (shrunken horizontally) when the display becomes smaller 3
Steps to Reproduce
Steps to reproduce the behavior:

Go to the ColorStack Member Profile website
Click on 'Profile' on the sidebar
Decrease the width of your browser
See error.

Expected Behavior
The user's profile picture will be shrunken horizontally and become vertically off-center.
Screenshots

Additional Context
This occurs on the page where you edit your personal profile. This bug was detected on Firefox on my MacBook.

@RiosNicholas this might be a MacOS issue because on my windows, my pfp does not get distorted

I can take this on :)! It would be my first open source contribution

I was planning to take it on, but I'm having issues getting my environment set up 😅. Please do work on it, though

I tried to replicate this on MacOS with chrome and it seems to be working fine for me. Are there any additional steps to simulating the bug?

Interesting... I'm on Firefox, so I wonder if the browser is part of the problem 🤔

I tried to replicate it on Firefox and also started having issues with the build. Interesting, maybe there's something deeper? I couldn't get past the OTP screen.

You're also having issues setting up your environment, or what do you mean? I just checked again and am still able to replicate the error 🤔

@ciaracade I just assigned this ticket to you! Can't wait for you to get your first open-source contribution in!
Just a note - I think to reproduce this issue, you have to make your headline long enough to take 2 or more lines. It seems like that's causing the profile picture to shrink!

Update for @ciaracade this issue now extends to safari users when I hit profile 😔

Hey @ciaracade, are you still planning to work on this? I might take a look this weekend

Hey @ramiAbdou, should I wait to proceed since I'm not assigned to this issue? Thanks

@RiosNicholas Just confirmed with Ciara that you can take this one. Reassigning to you!

I'm doing some investigating right now, and while I was unable to reproduce the bug in the dev environment, I found what is causing the issue. I noticed that having a long word (without spaces) as a header didn't result in word breaks. Rather, it would just proceed beyond the viewport width.

Initially, I didn't think implementing a word-break tailwind class into the Text component would be important, but I decided to do the same thing in the live member profile. I found that having long words will eventually cause the profile picture to disappear because there isn't space for the text to overflow in the deployed version's headline.

This draws me to conclude that the solution may include implementing a word wrap (word-break) prop in the Text component, and I'll work on that. @ramiAbdou

Update: I solved it without having to update the compone, but my commits went into the wrong PR 😭 . I'll fix

authors: JustinhSE RiosNicholas ciaracade ramiAbdou

repo: colorstackorg/oyster


text:
Feature proposal: Inline assets in HTML output

(I already made an issue with simplecov-html, see there.)

+1

Looks like this has been done, as a formatter: grosser/simplecov-single_file_reporter

I think this is purely simplecov-html related so an issue over there should be enough :)

authors: PragTob eins78 rthbound

repo: colszowka/simplecov


text:
Tests for warnings

@colszowka , regarding #451, I found a partial solution. It seems Ruby warnings are issued without consistency:

From Ruby code, using Kernel#warn
From C code, using rb_warn()
Printing to $stderr directly in some cases

If warnings are issued within an example block, this PR will detect and fail the test. As  bonus, it will not fail tests if warnings were issued by code at some dependency outside the project.
Warnings issued outside example blocks, for example with a require, will not be detected though. The solution, would be to use the same technique, but inside the Rakefile perhaps.
Anyway, partial solution is better than nothing :-D

I'll fix Ruby 1.8


text:
How to handle update by multiple users at the same time

As a engineer
I don't want to lose information if a colleague edits at the same time
So that the correct data is in de store
Background:
If 2 engineers are editing the same configuration the last one saving the configuration will now win.
The changes of the other engineer will be lost in the latest version. But in the history it's will be there.

Maybe check later if this is a problem, because all versions are saved, zo no data is lost.

Each save creates a new version in the database, so both versions can be compared.
If optimistic locking is needed a separate issue can be created to solve this in the BaseX/PostgreSQL implementation.
The solution is probably to check if the version passed in the Header is the latest version.

repo: com-pas/compas-scl-data-service


text:
feat: have routine that takes us from consensus to blocksync

Feature Request
Summary
We have recently noticed with going to faster block speeds, when the network is spammed and validators begin to not sign in time, they continue to not sign but still make block progress. Dev correctly pointed out that the reason for this is that once nodes are in consensus mode, there is no check to bring them back to blocksync if they start to fall behind. This results in a large set of validators being behind indefinitely / until they restart their nodes. Similarly to how we have a IsCaughtUp check in blocksync, we should also be able to check this in consensus and then switch back to block sync in the event a peer is maybe like, 3 blocks ahead of us or higher.
I made a few attempts to implement this, I was able to get the node to realize it was behind when in consensus mode, but it wouldn't switch back to blocksync properly and would just get stuck. I think I am just missing some context in the interworking of these reactors, I imagine the solution shouldn't be too involved.

This is similar to https://github.com/tendermint/tendermint/issues/129. Notice how old this issue is!
Also related to https://github.com/tendermint/tendermint/issues/4644

I think back then it was a nice to have, but now is becoming much more relevant with chains moving to 1-2 seconds blocks!

It is not simple to solve, although we are aware about some workarounds. We need to:

Identify that we are lagging behind, now the consensus reactor does that
Stop the consensus logic, the state
Run block sync until catching up
Restart consensus logic, this part already exist

Ideally, we would need some component that observes how the block heights evolve on the surrounding peers, e.g., from the consensus reactor. This component should be able to trigger block sync to do its job, while keeping consensus alive, but in a way that we can tell consensus to skip a height because we have got the block for that height from somewhere else (e.g., blocksync).
So, ideally it is still consensus running. But we use block sync to fetch block and commit sets faster. And provide then to consensus, so that the operation of the node, from the user, app, etc. point of view is the same. It is just that we are helping consensus to move faster by downloading blocks via block sync.

The documentation we have for block sync is: https://github.com/cometbft/cometbft/blob/main/docs/explanation/core/block-sync.md
Last sentence:
"If we're lagging sufficiently, we should go back to block syncing, but this is an open issue."

I made a few attempts to implement this, I was able to get the node to realize it was behind when in consensus mode, but it wouldn't switch back to blocksync properly and would just get stuck. I think I am just missing some context in the interworking of these reactors, I imagine the solution shouldn't be too involved.

@czarcas7ic Do you have any of the diffs from when you worked on this? Can you share any code or ideas around how you got the node to realize when it was behind?

@wllmshao I think the branch I was working off was https://github.com/osmosis-labs/cometbft/commits/adam/testing-switch-back-to-blocksync/
It honestly feels like so long ago that I essentially lost whatever ctx I had when I was working on it. Hopefully something from that is salvageable, but I wouldn't get your hopes up :(

I ended up working on a POC of this that seemed to work completely for an e2e test where a single node disconected and reconnected. Although the POC is severely limited, I think it captures the basic things that need to be done to switch from consensus to blocksync and back.
You can find the code and discussion of the POC linked in the issue here: https://github.com/skip-mev/cometbft/issues/3#issuecomment-2450931544

authors: cason czarcas7ic hvanz wllmshao

repo: cometbft/cometbft


text:
test(p2p/pex): fix TestConnectionSpeedForPeerReceivedFromSeed flacky test

Increased the timeout from 1 to 3 seconds.
Almost never fails locally, but fails on the GitHub CI (e.g., https://github.com/cometbft/cometbft/actions/runs/10111986972/job/27965072382?pr=3411).

PR checklist

[x] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[x] Title follows the Conventional Commits spec

This does not solve the problem...


text:
Allow browser launch template to be specified per-profile

Suggested by Robert in our Community Slack:
Currently there isn’t a way to define a browser launch template within the AWS profile file, but this seems like a good feature we could add.
[profile example]
granted_aws_console_browser_launch_template = "flatpak run org.mozilla.firefox --new-tab {{.URL}} --profile work"

Duplicate of #771

repo: common-fate/granted


text:
5.1.7 FPT_VDP_EXT.1, Tests

It seems that this is something that could at least be partially tested by the evalutors; since the list of legal values is required to be specified, it seems that it would provide more assurance than just attestation if the evaluators had to perform one test per interface where there was a clear illegal value to show that the illegal value was not accepted.
Attestation provides no assurance.  Testing at least provides some assurance.

Let's see what the TC says. I would prefer a test.

Let's see what the TC says. I would prefer a test.

Agree. I added a test and removed the attestation.

repo: commoncriteria/virtualization

license: Unlicense


text:
Add ScrollView in About activity.

Description
Fixes #1177
A ScrollView is added so that content does not get cut-off for mobile phones with small screen size.
Screenshots showing what changed

Codecov Report

Merging #1178 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff           @@
##           master   #1178   +/-   ##
======================================
  Coverage    3.85%   3.85%
======================================
  Files         125     125
  Lines        5757    5757
  Branches      567     567
======================================
  Hits          222     222
  Misses       5520    5520
  Partials       15      15

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 45aab1c...00405d1. Read the comment docs.

authors: codecov-io knight-shade

repo: commons-app/apps-android-commons


text:
Feature/nearby io fix

Nearby crashes when Home [From the Nav Drawer] is pressed while nearby is still loading data.
Fixes #{#1846, Nearby crashes when Home [From the Nav Drawer] is pressed while nearby is still loading data }
Description (required)
While nearby is still loading, and home from the nav drawer is clicked, the app used to crash with InterruptedIOException.
{Describe the changes made and why they were made.}
For fixing the same, I have catched the exception  the function getFromWikidataQuery(curLatLng, lang, radius) in NearbyPlaces. The cause of the exception could be anything, primarily IO. After catching the exception I returned null for places, for which I had to handle null for places in loadAttractionsFromLocation(LatLng curLatLng).
Tested on {27 & samsung s7}, with { ProdDebug}.
Tested for stability, Nearby no longer crashes in the mentioned scenario.
Screenshots showing what changed (optional)
NA

Codecov Report

Merging #1847 into master will decrease coverage by <.01%.
The diff coverage is 0%.

@@            Coverage Diff            @@
##           master   #1847      +/-   ##
=========================================
- Coverage    3.61%   3.61%   -0.01%
=========================================
  Files         192     192
  Lines        9838    9842       +4
  Branches      871     871
=========================================
  Hits          356     356
- Misses       9456    9460       +4
  Partials       26      26

Impacted Files
Coverage Δ

...a/fr/free/nrw/commons/nearby/NearbyController.java
0% <0%> (ø)
:arrow_up:

.../java/fr/free/nrw/commons/nearby/NearbyPlaces.java
0% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e458421...efaeb7f. Read the comment docs.

Looks good to me, happy to merge once it has been tested.

@misaochan I have personally tested this 4-5 times. Is there any specific test case you want me to perform ?

@ashishkumar0207 I was referring to additional tests required as part of our project maintenance policy - https://github.com/commons-app/apps-android-commons/wiki/Project-maintenance . All PRs except for versioning/changelogs (including my own!) have to be tested by someone else before they are merged. :)

@misaochan Got you, ofcourse

authors: ashishkumar468 codecov-io misaochan


text:
Image stretching in lollipop 5.1 version

It is creating stretched image in lollipop 5.1 version.
It will good if you can fix it.

I cannot help you with this limited amount of information. Quoting the documentation:

Be certain to include complete steps for reproducing the issue.

In this way it is getting stretched.
If  I am using this in version other than the lollipop 5.1 , it works great.

I repeat: I cannot help you with this limited amount of information. Quoting the documentation:

Be certain to include complete steps for reproducing the issue.

This would include:

The source code to a project that reproduces the problem, or an indication of how the problem can be reproduced using one of this project's demo apps

Steps to use that code to reproduce the problem

The specific devices (Build.MANUFACTURER and Build.PRODUCT values) that reproduce the issue given the aforementioned code and steps

You can easily reproduce in any phone by using the demo code of your project only in any phone model that have android version 5.1 like (any android one phone..).

I cannot reproduce the effect on a Nexus 4 (running Android 5.1) using the demo/ app. Both the previews and the resulting pictures (in single-shot mode) are fine. It could be that these are not the steps that you expected, but you decided not to provide complete steps to reproduce the problem, and so I had to guess.

Okay, let me check once for model other than android one. It may be specific to those devices only.
Any way thanks for the help.

authors: amitshekhariitbhu commonsguy

repo: commonsguy/cwac-camera


text:
BUG: Fix crash upon selecting files with DICOM browser import dialog

This closes #980.
Using my Windows machine and testing with 3D Slicer I was able to successfully import a file to the DICOM browser.

Previously it would crash upon selecting the "Import" button of the native dialog.

Fixes https://github.com/commontk/CTK/issues/980

Yes, that looks like the fix.
How about getting rid of all the Q_ASSERTS and replacing them with CTK_SOFT_ASSERT messages?  This could be a different PR.

Before merging I would suggest this gets tested on macOS as well for good measure. I only tested on Windows which is all I have available to me.

I don't think this issue depends on the OS.

authors: jamesobutler pieper

repo: commontk/CTK


text:
When I scan my library, new albums / artists aren't displayed directly

I actually need to reload the app, which interrupt the sound. Would it be possible to update directly?

It is technically possible and relatively easy to implement. Some changes should update the UI automatically, like when editing your library content (e.g. replacing an artwork) it should immediately update UI. However it cannot work if the socket proxy isn't working (the one on port 9001)

It actually works when the socket proxy is on.

repo: compactd/compactd


text:
keep unsent msgs correlated with conversation

Switching between conversations with an unsent message in the input box and the inputed msg stays constant. Expected behavior is for the input text to be part of the conversation, to be able to switch to another conversation and type a msg, and then switch back to the other conversation and return to the msg I was writing.

I'll take a PR. This is surprisingly hard.

authors: marcopeereboom raedah

repo: companyzero/zkc

license: isc


text:
Update CONTRIBUTING.md to be consistent with ci

This updates the developer instructions in CONTRIBUTING.md to be consistent with ci.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 52.42%. Comparing base (09360ac) to head (b3182ce).

Additional details and impacted files

@@           Coverage Diff           @@
##             main      #86   +/-   ##
=======================================
  Coverage   52.42%   52.42%
=======================================
  Files          15       15
  Lines         599      599
=======================================
  Hits          314      314
  Misses        285      285

authors: codecov-commenter mcbarton

repo: compiler-research/xeus-cpp


text:
feat(quic): support rustls provider aws-lc-rs

Also see quinn-rs/quinn#2056

What if two features enabled together?
What if the user would like to use another crypto provider?

What if two features enabled together?

The user has to set a default provider via CryptoProvider::install_default or rustls will complain.

What if the user would like to use another crypto provider?

quinn-proto does not support other providers.

https://github.com/actions/setup-go
Use setup-go to specify a go version and run clippy CI.

What if the user would like to use another crypto provider?

quinn-proto does not support other providers.

Pretty sure it does, but not with any built-in support.

Pretty sure it does, but not with any built-in support.

@djc Well, you're right. There's only a little problem here: if neither rustls-ring or rustls-aws-lc-rs enabled, ring will not be imported in quinn-proto. If the user want to use a third-party provider, they need to at least introduce ring or aws-lc-rs together into the project.

Pretty sure it does, but not with any built-in support.

@djc Well, you're right. There's only a little problem here: if neither rustls-ring or rustls-aws-lc-rs enabled, ~ring~rustls will not be imported in quinn-proto. If the user want to use a third-party provider, they need to at least introduce ring or aws-lc-rs together into the project.
EDIT: ring -> rustls

That's fair. We could conceivably add a rustls-no-provider feature that doesn't pull in a crypto provider via rustls. The problem is that quinn-proto itself needs a bunch of non-TLS crypto which is currently also implemented using ring/aws-lc-rs so I guess, from that point of view @AsakuraMizu is correct in stating that Quinn does not currently support other backing cryptographic libraries.

authors: AsakuraMizu Berrysoft djc

repo: compio-rs/compio


text:
add deploy contract instruction

archetype deploy_contract

/*
archetype dummy (owner : address)

var n : nat = 0

entry default(i : nat) {
  called by none
  effect {
     n := i
  }
}
*/

import "./dummy.arl" as dummy

entry exec () {
  var okh : option<key_hash> = none;

  var oaddr : option<address> = none;
  transfer 0tz to deploy dummy with (selfaddress) at oaddr

  var op_addr : (operation * address) = mkcontract(0tz, okh, dummy, (selfaddress))

}

Deprecated see #343

repo: completium/archetype-lang


text:
packcheck-remote CLI

[x] Should print help when no arguments are specified
[x] In the help it is not clear what force mode means
[x] In the help, not clear which arguments are optional which are mandatory, which take arguments etc
[ ] The way of passing arguments is not the same as standard unix CLI, it uses option=value format only.

Moved to #66

authors: adithyaov harendra-kumar

repo: composewell/packcheck


text:
Add a licence and README

Licensing software is always good and a README might be helpful for the ol' bus factor that you're pretty heavily involved with, @qaisjp...

I would say an MIT licence would probably be good.

(See community checklist)

License added in 7e9c6c924cf7a8e4ffa8b21d2c3c41d549c0d69c

authors: neanias qaisjp

repo: compsoc-edinburgh/bi-drive-link


text:
[WIP] Added organizations that can contain other organizations

Organisaties samenvoegen en extra toevoegen onder nieuwe naam (de huidige organisaties moeten ook blijven)

[ ] Alle organisaties samenvoegen in een totaal maat.
[ ] Het Nordwin College Niveau 3-4, Het ROC FRIESE POORT en Het Nordwin College Niveau 1-2 samenvoegen en Het Buro noemen.

@emerencia , wat vind jij hiervan? Ik weet dat je geen fan bent van de has_many through / many to many relatie, maar dit leek me de netste oplossing. Of willen we dit op een andere manier oplossen?
Ik heb het nog niet volledig uitgewerkt omdat ik eerst graag jouw mening wou horen.

@frbl Deze kan nu weg toch? Of is #349 geen vervanging hiervan?

authors: emerencia frbl

repo: compsy/vsv


text:
Fix skipped unit tests on macOS / M1

They are skipped in the CMakeLists.txt. Grep on Darwin.

Fixed as part of #474

repo: computationalgeography/lue


text:
WIP add Hamza Adrat paper

waiting for DOI and ISSN

@jchiquet merge if OK
sorry for the large diff, autoformatting...


text:
what happens if we try to download (and later rerun) an ongoing CI run logs?

i.e. will we just get a partial result and then upon rerun skip that run completely?

@yarikoptic Any actions that are still in progress are skipped when downloading, and the timestamp saved in the statefile will be before the earliest in-progress action, so on a subsequent run, the skipped action (if it's completed by then) will have its logs downloaded, and any actions after it that were downloaded previously will be skipped due to there already being logfiles saved locally.

Great, thank you

authors: jwodder yarikoptic

repo: con/tinuous


text:
CONAN_BASE_PROFILE_BUILD is incorrectly passed to docker

Changelog: (Fix): This is a draft PR that is just a work in progress trying to pass CONAN_BASE_PROFILE_BUILD to docker
Note: This is a draft PR as it currently doesn't work and includes lots of print-debugging (some of which I don't even see in the logs). It is just meant to serve as a point of discussion on #595. It will either be cleaned up or closed.
A failing example of an actual run that uses this patched cpt can be found here: https://github.com/jmarrec/conan-openstudio-ruby/runs/4852398264?check_suite_focus=true

references https://github.com/conan-io/conan-package-tools/issues/595
[ ] If the issue has missing info, explain the purpose/use case/pain/need that covers this Pull Request.
[x] I've read the Contributing guide.
[x] I've followed the PEP8 style guides for Python code.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Is this PR obsolete?

@Croydon Non functional rather, but the need is still here, though I've worked around it since then. I'm fine closing if no one has the bandwith to help out finish it though.

I don't understand the issue currently, the CI logs got deleted in the meantime and the references issues closed as solved 🤔

authors: CLAassistant Croydon jmarrec

repo: conan-io/conan-package-tools


text:
log same events in rete v2

fix toggle minigraph log event
add logging of connection and node creation and removal

@bacalj this is the 3rd PR that I thought you could review. However it has a failing jest test, so I need fix that first. I'll add you as a reviewer when the tests are all set. Since this is just logging changes, it isn't user visible so it is OK if it takes longer to get this merged.

Alright this is ready for review now. The jest test failure was a random network issue.


text:
What's the difference between VividStore & CommunityStore, which one do we use?

Haven't really been following VividStore, but now I see this one that looks much alike. Is VividStore going it's own way, or is that project being left behind or what happened here?
(https://github.com/VividWeb/vivid_store)
So best question would be, which one to use of the 2?

Ryan has rewritten pretty much most of the code by now. I can't speak for him, but as far as I know he and others needed some changes that didn't happen to VividStore so that community store was born.
Unlike VividStore we have several people who have access to this repo. In the end it's more like a community project, eventhough Ryan does pretty much most of the work at the moment. I switched to community store mostly because it's more active, pull requests get reviewed and merged much faster and new features get added more frequently.
I've seen some changes in VividStore lately, guess Michael has a bit more time again.

Remo has summarised that quite well.
Feature wise the two are pretty similar and achieve the same goal, but as Remo has mentioned the code underlying most of it has been quite heavily rewritten - pretty much every file has been reworked substantially. It really just shares some history, but they're very different code bases now.
The significant feature differences:

Blocks and single pages are by default marked up with bootstrap styling, so it's directly compatible with Elemental. It doesn't have it's own style as such.
Code wide it mostly uses Doctrine for models, with entity relationships defined more heavily.
Dashboard pages are styled as per default concrete5 dashboard styles
Shipping methods are structured differently, with each method able to return multiple 'offers'
Product blocks output Structured Data (for Google to pick up)
There is a Duplicate Product feature
There is a compatible Stripe add-on available
There are different/more event hooks
Community Store and the other components in the organisation are MIT licensed.

It's simply not for me to suggest which one to use. One is available on the marketplace and supported by Vivid, this one is a community driven project that lives on github and is intended more as a platform for developers.
If you haven't seen it already I've tried to document the project goals (and other important aspects) in the wiki - https://github.com/concrete5-community-store/community_store/wiki

OK, got it. But that means some still stay/stick with VividStore (as developers), right? Wouldn't it make more sense to have as much people as possible on the same ship, instead of taking 2 directions?
eCommerce for 5.7 is a good thing, but even I as a developer got confused what happened here. I'm not following VividStore at all, but just came across a tweet of Ryan and thought I should take a look (since when I looked at VividStore last time, I didn't see much commits/activity). If a project has a lot of inactivity, I lose faith in it. It seems you guys are trying to build something active here, but have the others been informed as well about this one? A lot of time may be going in 2 different projects, where it could be spend into just "this" one.

I certainly agree that it's important to focus energies, and I didn't take the decision to rework the store lightly.  But as the goal of this project is arguably different from that of Vivid Store, the codebase is quite different and the licenses are different, I don't think there'll be any developmental overlap.

authors: Mesuva Remo ramonleenders


text:
Build thumbnails over ajax

This pull request:

Disables the old thumbnail middleware
Adds a route that will try to build a thumbnail one at a time
Makes it so that a single thumbnail will not be built by more than one user
Automatically will update 404'd images in the page as the thumbnails get built

I appreciate what we're doing here, but I'm wondering if we shouldn't just simplify this whole process. What instead of using anything like this, in the BasicThumbnailer class we computed the ID of the thumbnail we're trying to load, and instead we just created a new route that is
/ccm/system/thumbnail/{thumbnail_id}
(I don't know what the actual route should be)
and we actually just put that into the generated HTML? This is classically how PHP thumbnailers have worked.
<img src="/index.php/ccm/system/thumbnail/1238718">

And this would just return the cropped thumbnail. Like a classic boring dumb old PHP thumbnailing script.
The system could still lock transactionally when building/retrieving the thumbnail, and it could still use the table to determine whether the thumbnail was actually built. But you could dramatically simplify everything and not worry about whether you were running javascript when you were logged in, including this fancy src replacer on every page, etc...

@aembler doing that has a couple problems:

Many times our thumbnailer is incapable of generating thumbnails, what you recommend would lead to us infinitely trying to build it.
Since we have no way to replace these urls in the cache, that would add huge amounts of load to NGINX/Apache and PHP since all images would flow through the PHP process and would need to start and stop session and all that. I wouldn't recommend we do this for our larger clients.
Sites with large amounts of traffic will be building a single thumbnail on every session that views the page while the thumbnailing is happening, this causes issue for some of our clients that have many page views each second.

In my opinion, we should fix the things pointed out by @Mnkras and move forward with this until we can come up with a plan of action that answers to all of these concerns.

@Mnkras

Does the toolbar.js work for people that are not logged out?

No, @aembler recommended we move this to the header_required but I'm not so sure about that since we don't force jQuery to be loaded. I'm happy to make that change if that's what we think we should do. Having this run on the front end would probably be great.

I think 1 and 3 are pretty easily fixable – number 2 is certainly a fair point. Some thoughts:

I think we should do more than 1 at a time, and we should put that value into a config variable that can be changed. Half the complaints we're getting are from people using image galleries...if you've got 40 images on a page 1 at a time is gonna be rough. We used to do all 40 at one time in that situation (which obviously isn't great either.) I'm not sure what the right number is but it has to be more than 1.

Mnkras is right that we can't rely on the ajax completing in order to handle the thumbnails. They have to eventually be built. For example, what if someone sets up a gallery but the gallery is in a block that doesn't show any view layer in edit mode? So someone sets up a gallery with 40 images (i love this 40 image example for some reason) approves the edits and closes their browser tab? Maybe a couple of the images get built but then the gallery is basically broken until someone else logged in visits the page and the magic happens behind the scenes? I think the goal with the async JS is to mitigate the broken images happening until the page got reloaded.

Honestly this feels like a job that runs and does 10 thumbnails at once and runs on every page load (which I realize is basically what the middleware was) and then the async is just set up to run in edit mode and do automatic image replacement in order to combat the weirdness of the 404 images.

I wasn't sure the best way to add to this so I superseded it with https://github.com/concrete5/concrete5/pull/5716 (which basically branches from your ajax thumbnails and makes them work when not logged in, updates the migration, removes some debug code, etc...)

authors: KorvinSzanto aembler


text:
Update Form.php

TAB indention -> spaces
Check these before submitting new pull requests

[ ] I read the guidelines for contributing linked above

[ ] PHP-only files follow our coding style; in order to do that use php-cs-fixer - http://cs.sensiolabs.org/ - as follows:
php-cs-fixer fix --config=<webroot>/.php_cs.dist <filename>

If all the above conditions are met, feel free to delete this whole message and to submit your pull request, and... Thank you, your help is really appreciated!

Two new functions for form building:
public function radioList($key, $optionValues, $valueOrMiscFields = '', $miscFields = [])

public function checkboxList($optionValues = [], $miscFields = [])

The clarity of the contained select() function tempted me to look for something similar to easily set up a list of radio buttons or a related list of checkboxes. As I didn't find anything I developed the two functions.
Example for radioList:
echo
$form->label('rss', t('Provide RSS Feed'), [ 'class' => 'radioLine-3' ]),
$form->radioList('rss', [
        0   => t('No'),
        1   => t('Yes'),
    ], [
        'checked'   => is_object($rssFeed) ? 1 : 0,
        'class'     => 'radio radioLine-3',
        'inputClass'=> 'rssSelector',
    ]);

Example for checkboxList:
echo
$form->label(null, t('Other Filters')),
$form->checkboxList( [
        'displayFeaturedOnly'   => [
            'label'     => t('Featured pages only.'),
            'disabled'  => !is_object($featuredAttribute),
            'help'      => t('(Note: You must create the "is_featured" page attribute first.)'),
            'checked'   => $displayFeaturedOnly == 1,
        ],
        'displayAliases'        => [
            'label'     => t('Display page aliases.'),
            'checked'   => $displayAliases == 1,
        ],
        'ignorePermissions'     => [
            'label'     => t('Ignore page permissions.'),
            'checked'   => $ignorePermissions == 1,
        ],
        'enableExternalFiltering' => [
            'label'     => t('Enable Other Blocks to Filter This Page List.'),
            'checked'   => $enableExternalFiltering,
        ],
    ], [
        'class'     => 'checkbox',
        'inputClass'=> 'otherFilters',
    ]);

...due to Travis error


text:
Update/art 1.9.0

Checklist

[X] Used a personal fork of the feedstock to propose changes
[X] Bumped the build number (if the version is unchanged)
[X] Reset the build number to 0 (if the version changed)
[X] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[X] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe:

Recipes should usually depend on matplotlib-base as opposed to matplotlib so that runtime environments do not require large packages like qt.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


text:
Mention b3sum for command line Blake3 tool

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

I was initially concerned that the python blake3 package had taken the obvious name for the reference Blake3 implementation https://github.com/BLAKE3-team/BLAKE3 - although on further investigation that package is called b3sum after the command line tool. This PR adds a brief summary to preempt this potential end-user confusion.

@conda-forge-admin, please rerender

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found some lint.
Here's what I've got...
For recipe/meta.yaml:

The license item is expected in the about section.

authors: conda-forge-admin peterjc


text:
Updated to v0.1.3

Updated to v0.1.3. Relocated summary, moved description to 1 line. Small fixes for pip based installer resulted in new SHA256 hash.

Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

There are too few lines.  There should be one empty line at the end of the file.

@conda-forge-admin please rerender

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

authors: conda-forge-linter wouterpotters

repo: conda-forge/castorapi-feedstock


text:
@conda-forge-admin, please update version

Additional comment:
No response

@conda-forge-admin, please update version


text:
0.5.5 hf99513

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes https://github.com/conda-forge/emd-feedstock/pull/6

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@conda-forge-admin, please rerender

@conda-forge-admin, please rerender

Looks good to me!

Amazing - thank you @larsoner & @hoechenberger. I'd spotted some deps were a bit out after my last minor release but hadn't got around to sorting it. Happy to merge this now.
I'm planning to improve how dependencies are handled and checked in emd in the next release, likely coming this week. Hopefully if the version changes here are maintained this wouldn't affect things on your side?

PS - I didn't realise EMD was distributed in the MNE installer... very cool!
Please let me know if any other tweaks or changes come up.

Hopefully if the version changes here are maintained this wouldn't affect things on your side?

Indeed things should be fine as long as no crazy pinning happens in the process -- then conflicts can occur with other packages we want that have pins of their own.

authors: AJQuinn conda-forge-linter hoechenberger larsoner

repo: conda-forge/emd-feedstock


text:
Updated dependencies

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.
I do have some suggestions for making it better though...
For recipe/meta.yaml:

ℹ No valid build backend found for Python recipe for package gandlf using pip. Python recipes using pip need to explicitly specify a build backend in the host section. If your recipe has built with only pip in the host section in the past, you likely should add setuptools to the host section of your recipe.
ℹ noarch: python recipes should usually follow the syntax in our documentation for specifying the Python version.

For the host section of the recipe, you should usually use python {{ python_min }} for the python entry.
For the run section of the recipe, you should usually use python >={{ python_min }} for the python entry.
For the test.requires section of the recipe, you should usually use python {{ python_min }} for the python entry.
If the package requires a newer Python version than the currently supported minimum version on conda-forge, you can override the python_min variable by adding a Jinja2 set statement at the top of your recipe (or using an equivalent context variable for v1 recipes).

ℹ PyPI default URL is now pypi.org, and not pypi.io. You may want to update the default source url.

This message was generated by GitHub Actions workflow run https://github.com/conda-forge/conda-forge-webservices/actions/runs/11975892451. Examine the logs at this URL for more detail.

Hi! This is the friendly conda-forge automerge bot!
I considered the following status checks when analyzing this PR:

linter: passed
azure: failed

Thus the PR was not passing and not merged.

authors: conda-forge-admin sarthakpati

repo: conda-forge/gandlf-feedstock


text:
Refresh pinnings

Refresh the pinnings to match the most recent ones.
Drop libtool from requirements/run.
Package the license file.
Fix selector spacing in requirements.
Re-render with conda-smithy 2.4.2.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

authors: conda-forge-linter jakirkham

repo: conda-forge/graphviz-feedstock


text:
@conda-forge-admin,please add bot automerge

@conda-forge-admin,please add bot automerge

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added bot automerge in conda-forge/hepunits-feedstock#2.

authors: chrisburr conda-forge-linter

repo: conda-forge/hepunits-feedstock


text:
Split libcxx and libcxxabi

Checklist

[ ] Used a fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The top level meta keys are in an unexpected order. Expecting ['package', 'source', 'build', 'requirements', 'test', 'outputs', 'about', 'extra'].

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

authors: conda-forge-linter isuruf

repo: conda-forge/libcxx-feedstock


text:
MNT: Re-render the feedstock [ci skip]

Hi! This is the friendly conda-forge-admin automated user.
I've re-rendered this feedstock with the latest version of conda-smithy (1.5.3) and noticed some changes.
If the changes look good, then please go ahead and merge this PR.
If you have any questions about the changes though, please feel free to ping the 'conda-forge/core' team (using the @ notation in a comment).
Remember, for any changes to the recipe you would normally need to increment the version or the build number of the package.
Since this is an infrastructural change, we don't actually need/want a new version to be uploaded to anaconda.org/conda-forge, so the version and build/number are left unchanged and the CI has been skipped.
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The recipe license should not include the word "License".

Should be fine to merge. Lint can be dealt with in a subsequent PR.

It's the friendly automated conda-forge-admin here again.
Just to let you know, I've updated this PR so that it has the latest render from conda-smithy (version 2.0.0).
If there are no problems with it, please consider merging this PR.
If there are concerns about it, please ping the 'conda-forge/core' team (using the @ notation in a comment).
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I wanted to let you know that I linted all conda-recipes in your PR (recipe) and found some lint.
Here's what I've got...
For recipe:

The recipe license should not include the word "License".

repo: conda-forge/matplotlib-venn-feedstock


text:
MNT: Re-rendered with conda-smithy 1.6.0

For python 3.6

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

Please make sure you have conda-smithy 1.7.0 installed in root.

Restarted Travis CI and triggered CircleCI to start.

I have "defaults" given priority over the conda-forge channel, so it would only upgrade to conda-smithy 1.6.0 without an explicit -c conda-forge. Re-rendered with 1.7.0

Could you temporarily reverse the order so conda-forge comes first and then re-render again?
Note: We plan to ignore users channel settings in the future w.r.t re-renderings. Unfortunately all version of conda-smithy up to this point have used the local settings from the user.

superseded by #5

On 09.02.2017 03:20, Matthew Harrigan wrote:

superseded by #5 https://github.com/conda-forge/mdtraj-feedstock/pull/5
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/conda-forge/mdtraj-feedstock/pull/4#issuecomment-278527486,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAKZL8Os4ntFtyJ93ABAXTeV9txYnwpFks5rangGgaJpZM4LdB20.

#6 is also the same conda smithy version as #5.

authors: conda-forge-linter jakirkham marscher mpharrigan

repo: conda-forge/mdtraj-feedstock


text:
[ci skip] adding user @mukhery

Hi! This is the friendly automated conda-forge-webservice.
I've added user @mukhery as instructed in #31.
Merge this PR to add the user. Please do not rerender this PR or change it in any way. It has [ci skip] in the commit message to avoid pushing a new build and so the build configuration in the feedstock should not be changed.
Please contact conda-forge/core to have this PR merged, if the maintainer is unresponsive.
Fixes #31

Resolved with #33

authors: conda-forge-admin mukhery

repo: conda-forge/mgrs-feedstock


text:
Update meta.yaml

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

@conda-forge-admin, please rerender

@sugatoray can we review and merge it.

repo: conda-forge/nlptest-feedstock


text:
@conda-forge-admin, please add user @raimis

[x] I read the conda-forge documentation and could not find the solution for my problem there.

Issue:

Environment (conda list):

$ conda list

Details about  conda and system ( conda info ):

$ conda info

Hi! This is the friendly automated conda-forge-webservice.
I just wanted to let you know that I added user @raimis in conda-forge/openmm-torch-feedstock#14.

authors: conda-forge-linter raimis

repo: conda-forge/openmm-torch-feedstock


text:
Add dev and doc URLs.

Checklist

[ ] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-webservice.
It appears you are making a pull request from a branch in your feedstock and not a fork. This procedure will generate a separate build for each push to the branch and is thus not allowed. See our documentation for more details.
Please close this pull request and remake it from a fork of this feedstock.
Have a great day!

This PR will be reopened from a fork.

authors: MaxJPRey conda-forge-linter

repo: conda-forge/pyaedt-feedstock


text:
MNT: Re-render the feedstock [ci skip]

Hi! This is the friendly conda-forge-admin automated user.
I've re-rendered this feedstock with the latest version of conda-smithy (1.3.2) and noticed some changes.
If the changes look good, then please go ahead and merge this PR.
If you have any questions about the changes though, please feel free to ping the 'conda-forge/core' team (using the @ notation in a comment).
Remember, for any changes to the recipe you would normally need to increment the version or the build number of the package.
Since this is an infrastructural change, we don't actually need/want a new version to be uploaded to anaconda.org/conda-forge, so the version and build/number are left unchanged and the CI has been skipped.
Thanks!

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

authors: conda-forge-admin conda-forge-linter

repo: conda-forge/pylint-django-feedstock