@aladar42 Thanks for your pull request. We don't really like to use wildcards. There are a lot of subdomains for unitn.it, so if you don't want to add them all individually, you can use (partial) in the ruleset name and only cover the ones you want. See pull request #9729 for an example of a similar situation.

After you do this, I or another maintainer can take a look at what you have more closely.

I'm goign to take a look at it. Can't guarantee I'll do it all according to standards, though.

Any updates?

Please rebase your specific changes additions onto master, instead of including what appears to be a merge.

In other words, the "Files changed" tab above should show one file, not the current 883.

Sorry, I'm still pretty new to Git. Did I do it correctly now?

Additionally, if it's not too much trouble, what do you recommend as proper pull request in future commits?

The git situation looks okay now, thanks.

• Please rewrite your pull request like this:

<!--
	Invalid certificate:
		unitn.it
		cimec.unitn.it
		economia.unitn.it

-->
<ruleset name="Unitn.it (partial)">
	<target host="unitn.it" />
	<target host="www.unitn.it" />
	<target host="alumni.unitn.it" />
	<target host="centro3a.unitn.it" />
	<target host="www.centro3a.unitn.it" />
	<target host="cimec.unitn.it" />
	<target host="www.cimec.unitn.it" />
	<target host="gene.disi.unitn.it" />
	<target host="economia.unitn.it" />
	<target host="www.economia.unitn.it" />
	<target host="international.unitn.it" />
	<target host="knowtransfer.unitn.it" />
	<target host="phd-researchfellow.unitn.it" />
	<target host="pressroom.unitn.it" />
	<target host="unitrentosport.unitn.it" />
	<target host="webapps.unitn.it" />
	<target host="webmagazine.unitn.it" />

	<rule from="^http://(cimec\.|economia\.)?unitn\.it/" to="https://www.$1unitn.it/" />
	<rule from="^http:" to="https:" />
</ruleset>

This makes various improvements. Please let me know if you have any questions.

I'm not sure what you are asking in your question about proper pull requests, but you might like to look over our CONTRIBUTING.md.

Done.

Thanks, merged.